3 ms·
Yes, still sending the data is bad. Their docs [1] do not mention the generate command can take an exclusion argument [2]. Maybe the dot (.) for current direct
by turtleyacht 2y ago
Yes, still sending the data is bad.
Their docs [1] do not mention the generate command can take an exclusion argument [2]. Maybe the dot (.) for current directory will exclude everything, but that may defeat the purpose.
If the current directory were the default, and on first run an error message like "Please include files first" appeared--and users also must enter "Y" at a disclaimer message--they could explicitly opt in instead.
And that should be placed before the tar-n-upload piece.
The pull request was also merged before approval. Github branch protection could have been set to require at least one approver.
[1] https://devenv.sh/blog/2025/02/13/devenv-14-generating-nix-developer-environments-using-ai/ https://devenv.sh/blog/2025/02/13/devenv-14-generating-nix-d...
[2] https://github.com/cachix/devenv/blob/main/devenv/src/cli.rs#L197 https://github.com/cachix/devenv/blob/main/devenv/src/cli.rs...