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

Add pure-storepath-builtin experimental feature #12141

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Pandapip1
Copy link
Contributor

Motivation

No consensus has been found as to whether storePath should be considered pure or impure. As such, an experimental feature should be added to allow those that consider the function pure to use it as such.

Context

See #5868.

Heads up: This is my first code change to nix; it's likely I missed something from the contributing document (even though I did read through it), and I couldn't find any documentation about the process of adding an experimental feature in the repo (I was on an airplane with no internet and am just now submitting the PR).


Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

@Pandapip1 Pandapip1 force-pushed the pure-builtins-storePath branch 2 times, most recently from 4b003ec to af76797 Compare January 6, 2025 18:27
@Pandapip1 Pandapip1 force-pushed the pure-builtins-storePath branch from af76797 to 56b63e2 Compare January 6, 2025 18:34
@Ericson2314
Copy link
Member

Heh, I would support this. It is sort of "unsure whether to stabilize as de facto stability 2nd time as farce", but shrug.

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

Successfully merging this pull request may close these issues.

2 participants