-
-
Notifications
You must be signed in to change notification settings - Fork 17
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 use of meta = with lib;
#292
base: main
Are you sure you want to change the base?
Conversation
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.
LGTM!
@figsoda any feedback on this? |
677db11
to
3e5d674
Compare
Sorry that I forgot about this for so long. Is there a consensus on this? If not maybe I can create a poll or something like that |
In my experience it gets constantly nitpicked in nixpkgs, |
No, there is no consensus on removing it from |
with lib;
(see NixOS/nixpkgs#208242)with lib;
This comment was marked as outdated.
This comment was marked as outdated.
with lib;
meta = with lib;
NB as mentioned in NixOS/nixpkgs#292468 (comment), (TLDR |
Motivating discussion: NixOS/nixpkgs#208242
Additional context: NixOS/nixpkgs#292468, NixOS/nixpkgs#293767
Fixes #414.