-
-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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
nixpkgs manual: Remove obsolete warning #21117
Conversation
@lheckemann, thanks for your PR! By analyzing the history of the files in this pull request, we identified @vcunat, @edolstra and @dezgeg to be potential reviewers. |
NixOS/nix#815 has been merged, but the default nix version does not contain it yet. See #19761 (comment) |
Ah right, perhaps the warning should just be reworded then... |
PR NixOS#815 has already been merged
0192614
to
88c5899
Compare
Ping |
@domenkozar: any indication the situation has changed? I sadly still see 1.11.6 as the default, without that PR contained or picked. |
Ah, I'm sorry – you probably meant that the rewording should be done. |
Since 1.11.8 is now the default version and contains the commit in question, I'm guessing this can be merged? |
Right, even though the situation isn't ideal, as discussed in #24717. |
PR NixOS#815 has already been cherry-picked into the default nix version 1.11.8.
PR #815 has already been merged.