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

bump rayon dependency #1479

Closed

Conversation

geo-ant
Copy link
Contributor

@geo-ant geo-ant commented Jan 26, 2025

I've just bumped the optional rayon dependency to the most recent version (as of today). There are failing tests that were failing before I changed anything. Everything else passes as it should.

Copy link

@jonaspleyer jonaspleyer left a comment

Choose a reason for hiding this comment

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

If there is not conflict in specifying version = "0.10" I would prefer this, since it allows greater compatibility with other crates. It could also be an option to choose version = **">=1.6, <1.11"** for the same reason.

@Ralith
Copy link
Collaborator

Ralith commented Jan 30, 2025

1.6 is semver-compatible with 1.10, so this doesn't seem to serve any purpose.

@Ralith Ralith closed this Jan 30, 2025
@jonaspleyer
Copy link

1.6 is semver-compatible with 1.10, so this doesn't seem to serve any purpose.

Yes this is totally my fault. I thought we were takling about 0.XX versions. You are right!

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