Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Cargo.toml: Bump open-enum to latest version #6

Merged
merged 1 commit into from
Nov 20, 2023
Merged

Cargo.toml: Bump open-enum to latest version #6

merged 1 commit into from
Nov 20, 2023

Conversation

jinankjain
Copy link
Contributor

@jinankjain jinankjain commented Oct 12, 2023 •
edited
Loading

Fixes #2

@chris-oo
Copy link
Member

Looks like a build issue? might need to tweak some derives maybe.

@jinankjain
Copy link
Contributor Author

@chris-oo Looks like this commit which breaks things: kupiakos/open-enum@ddd167a

@jinankjain
Copy link
Contributor Author

I have a fix for the aforementioned issue in this PR kupiakos/open-enum#15

@chris-oo
Copy link
Member

Okay, once the corresponding open_enum fix is in, we can take this.

@jinankjain
Copy link
Contributor Author

Okay, once the corresponding open_enum fix is in, we can take this.

The issue is fixed in 0.4.1. I have updated the PR to use the latest version.

@chris-oo chris-oo enabled auto-merge (squash) November 20, 2023 18:52
@chris-oo
Copy link
Member

Thanks! Set to automerge.

@chris-oo chris-oo merged commit 328de1d into microsoft:main Nov 20, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@russell-islam russell-islam russell-islam approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

bump open-enum dep to 0.4.0
3 participants
@jinankjain @chris-oo @russell-islam