-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
add "instantiate" to glossary #7318
add "instantiate" to glossary #7318
Conversation
421c4d6
to
48ecff6
Compare
|
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/tweag-nix-dev-update-40/23480/1 |
6133237
to
3043e20
Compare
3043e20
to
ca65bf6
Compare
Took out all the unrelated parts (covered in other PRs), now this only adds a term to the glossary. |
I do still hope per #6507 we will drop "store derivation", but this is correct for now. If/when we do, we will just revise this. |
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.
I've used the term loosely for adding sources to the store, which is required for adding derivations to the store, as derivations reference sources. This usage might be controversial and/or not worth adding to the definition here, so +1 on the current change from me.
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
ca65bf6
to
c203942
Compare
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/2023-01-23-nix-team-meeting-minutes-26/25433/1 |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/nix-team-report-2022-10-2023-03/27486/1 |
add a minimal definition and refer to
nix-instantiate
.This work is sponsored by Antithesis ✨