-
Notifications
You must be signed in to change notification settings - Fork 0
chore(rust/hermes-ipfs): bump rust-ipfs for hermes-ipfs #422
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
Conversation
stevenj
left a comment
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.
approved pending updates to CI.
433ee5d to
8e3d9c6
Compare
|
could we wait with this branch until this feature would be merged into the main #338, because as I see a new fmt rules are applied here, which could be quit complicated fix merge conflicts. |
The hermes project needs this merged. Can we either do a single PR now to all cat-libs that only fixes format (and nothing else), as its been changed. OR revert the format changes so these PR's are not enormous? CC: @Mr-Leshiy @rafal-ch Its unreasonable to hold this back when it has nothing actually to do with #338 |
|
I agree we should revert the format changes, these should not be mixed together with the bumps. Working on it. |
This reverts commit 52a726e.
|
Changes to |
|
✅ Test Report | |
Description
Bumps
rust-ipfsin order to bump its dependencies.Old dependencies cause advisers to issue security warnings.