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

Blacklist: add no-auto-update option #84

Merged
merged 1 commit into from Oct 24, 2018

Conversation

Synthetica9
Copy link
Contributor

There is currently no way to have r-ryantm ignore a package explicitly, so here it is now.

@ryantm
Copy link
Collaborator

ryantm commented Oct 24, 2018

@Synthetica9 Thanks for the contribution! I would like to try to keep nixpkgs-update (the code) separate from (r-ryantm) a bot run by me that is running the code. Ideally, I'd like to make it easy enough to run the code that other people can do it too (without stepping on the toes of each other!).

Would you be open to changing the magic string to be "nixpkgs-update: no auto update"?

@Synthetica9
Copy link
Contributor Author

Sure, that works for me. (Related thought: maybe also add a magic string to bypass the blacklist for derivations where we know this to work?)

There is currently no way to have nixpkgs-update ignore a package
explicitly. This adds that, with the magic string
"nixpkgs-update: no auto update".
@ryantm ryantm merged commit 1b2eb79 into nix-community:master Oct 24, 2018
@ryantm
Copy link
Collaborator

ryantm commented Oct 24, 2018

Thanks; merged. Please note the fancy indentations you made will probably be wiped out because we typically use hindent to automatically format the entire file. That was a decision @jtojnar and I made to help sidestep indentation debates.

@Synthetica9
Copy link
Contributor Author

Synthetica9 commented Oct 24, 2018 via email

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.

None yet

2 participants