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.

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

call EnumIter::get using fully qualified syntax #2321

Merged
merged 3 commits into from
Oct 4, 2024

Conversation

jcreekmore
Copy link
Contributor

Bug Fixes

itertools version 0.13.0 added a get method that takes precedence over the EnumIter::get method. The EnumIter derive should be calling the method fully-qualified to ensure that the local version is called rather than the method that is provided by the Itertools trait. This was fixed in strum 0.26.x, but since this is a forked/adapted version of the EnumIter derive, it requires the same fix .

`itertools` version 0.13.0 added a `get` method that takes precedence
over the `EnumIter::get` method. The `EnumIter` derive should be calling
the method fully-qualified to ensure that the local version is called
rather than the method that is provided by the `Itertools` trait. This
was fixed in `strum` 0.26.x, but since this is a forked/adapted version
of the `EnumIter` derive, it requires the same fix .
@Expurple
Copy link
Contributor

Expurple commented Aug 8, 2024

Oh, so that's what has been preventing me from updating to itertools 0.13! Thank you.

@jcreekmore
Copy link
Contributor Author

Is there something I am missing or need to do to get a review?

@Expurple
Copy link
Contributor

For the last few months, maintainers pay very little attention to community issues/PRs :(

It's weird, given that they still release new versions, blog posts, etc. They're probably focusing on their own checklist for sea-orm 1.0 and then sea-orm 1.1.0 with sqlx 0.8, although it's hard to say for sure.

Copy link
Member

@billy1624 billy1624 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @jcreekmore, sorry for the delay! I will merge this today and publish a new RC with this.

@billy1624 billy1624 merged commit fb44229 into SeaQL:master Oct 4, 2024
32 checks passed
Copy link

github-actions bot commented Oct 4, 2024

🎉 Released In 1.1.0-rc.2 🎉

Thank you everyone for the contribution!
This feature is now available in the latest release. Now is a good time to upgrade!
Your participation is what makes us unique; your adoption is what drives us forward.
You can support SeaQL 🌊 by starring our repos, sharing our libraries and becoming a sponsor ⭐.

Copy link

🎉 Released In 1.1.0 🎉

Thank you everyone for the contribution!
This feature is now available in the latest release. Now is a good time to upgrade!
Your participation is what makes us unique; your adoption is what drives us forward.
You can support SeaQL 🌊 by starring our repos, sharing our libraries and becoming a sponsor ⭐.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants