-
-
Notifications
You must be signed in to change notification settings - Fork 72
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
Remove unused sha256 argument for Git dependencies #124
Conversation
This argument has been rendered unused since the transition to `builtins.fetchGit` for all Git dependencies. Since a minor release is coming soon, this breaking change should now be safe to roll out.
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.
Let's remove the wrapper in default.nix
which bundles nix-prefetch-git
with cargo2nix
too.
@trha Sure! I'll do that next. I also noticed that I can turn |
Since the `Owned` case has been eliminated in the previous commit, there is no need to use `std::borrow::Cow` for this field anymore.
This is no longer necessary, now that we use `builtins.fetchGit` everywhere.
Done! Would you mind re-reviewing, please, @trha? |
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.
Looks good!
Okay, merging this in. I imagine this should be okay, since we've been awaiting this transition for a long time now. |
Changed
checksum
field ofResolvedPackage
fromOption<Cow<'a, str>
toOption<&'a str>
.Removed
sha256
argument for Git dependencies.serde_json
dependency.prefetch_git
function.nix-prefetch-git
wrapper fromdefault.nix
.This argument has been rendered unused since the transition to
builtins.fetchGit
for all Git dependencies was made (#113). Since a minor release is coming soon, due to #119, this breaking change should now be safe to roll out.Closes #102.