-
Notifications
You must be signed in to change notification settings - Fork 260
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
Update our lock file and Rust nightly version #3103
Conversation
The Dalek crates got a new release fixing some build issues on nightly. This should get rid of those build issues.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, I can confirm this fixes the issue for me locally.
But not on the CI 🥲. I think we'll need to bump our nightly version as well. |
We'll pin our nightly version in the CI to match yours. |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #3103 +/- ##
==========================================
- Coverage 83.71% 83.70% -0.01%
==========================================
Files 224 224
Lines 23534 23535 +1
==========================================
Hits 19701 19701
- Misses 3833 3834 +1 ☔ View full report in Codecov by Sentry. |
The IndexMap::remove method has been deprecated, the documentation[1] on the method tells us that we can replace the usage of it with IndexMap::swap_remove: > NOTE: This is equivalent to .swap_remove(key), replacing this entry’s > position with the last element, and it is deprecated in favor of > calling that explicitly. [1]: https://docs.rs/indexmap/2.2.2/indexmap/map/struct.IndexMap.html#method.remove
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving ahead of time to unblock you.
The linking error[1] on Mac still isn't fixed. [1]: seanmonstar/reqwest#2006
The Dalek crates got a new release fixing some build issues on nightly. This should get rid of those build issues.
Link to the relevant issue: dalek-cryptography/curve25519-dalek#618.