-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
Switch to using the v2 resolver in the library #128724
base: master
Are you sure you want to change the base?
Conversation
Pinning the resolver to v1 was done in 5abff37 ("Explicit set workspace.resolver ...") in order to suppress warnings. Since there is no specific reason not to use the new resolver and since it fixes issues, change to `resolver = "2"` in `library/`.
rustbot has assigned @Mark-Simulacrum. Use |
v2 resolves some dependencies differently. Run `cargo update` to make sure everything is in sync.
461a006
to
bce5f2c
Compare
The job Click to see the possible cause of the failure (guessed by this bot)
|
This will need more of the fixes from #128359. |
☔ The latest upstream changes (presumably #128805) made this pull request unmergeable. Please resolve the merge conflicts. |
Pinning the resolver to v1 was done in 5abff37 ("Explicit set workspace.resolver ...") in order to suppress warnings. Since there is no specific reason not to use the new resolver and since it fixes issues, change to
resolver = "2"
inlibrary/
.Split from #128359, goes with #128722 that updates the root workspace.