Skip to content
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 github link to use nixpkgs instead of nixpkgs-channels. #46

Merged
merged 1 commit into from
Mar 30, 2021

Conversation

thequicksort
Copy link

Hi, thank you for all your work on this package, I have a minor addendum.
Currently, the generated github link points to the old nixpkgs-channel repository (deprecated), while still using nixpkgs commit hashes:

Screen Shot 2021-03-14 at 7 54 17 PM

The result is that the github link 404s.

image

This pull request updates the link to use nixpkgs proper as the repository. The correct link in this case would be https://github.com/NixOS/nixpkgs/commits/c5147860e23ed75ce9d40298c66b416c00be1167.

@garbas garbas self-requested a review March 30, 2021 21:38
@garbas garbas merged commit 575e6ab into NixOS:master Mar 30, 2021
@thequicksort thequicksort deleted the nixpkgs-channel-migration branch March 30, 2021 21:41
@thequicksort
Copy link
Author

Thank you all for the reviews and for merging! 🙏

@garbas
Copy link
Member

garbas commented Mar 30, 2021

@thequicksort thank you for the work 🥇

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants