-
Notifications
You must be signed in to change notification settings - Fork 552
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
chore(cargo): update dependencies to compatible versions #2009
Conversation
2fef61f
to
dc7b132
Compare
sorry but i would prefer separate commits and not a big one. Way easier to bisect |
One crate per commit? |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2009 +/- ##
==========================================
- Coverage 30.74% 30.42% -0.33%
==========================================
Files 51 51
Lines 19564 19638 +74
Branches 9452 9437 -15
==========================================
- Hits 6015 5974 -41
- Misses 7905 8015 +110
- Partials 5644 5649 +5 ☔ View full report in Codecov by Sentry. |
Yes please
Le sam. 6 janv. 2024, 00:48, liushuyu ***@***.***> a écrit :
… sorry but i would prefer separate commits and not a big one. Way easier to
bisect
One crate per commit?
—
Reply to this email directly, view it on GitHub
<#2009 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAFTBDSL7ZDGS7TLNIROPTTYNCGOHAVCNFSM6AAAAABBPCW2OCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZZGM4DQNJRGQ>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Done. |
Perfect! |
But it looks like you folded all the commits when merging (using that button on the web interface?). |
Sorry for not reading previous discussion first, do we need to revert this PR first? Cc @sylvestre |
This pull request updates all the dependencies to their latest compatible version (no API changes and no MSRV changes).
This should supersede most of the automated pull requests.