3 ms·
you missed the part this still sends all the data regadless
by 1oooqooq 2y ago
you missed the part this still sends all the data regadless
- turtleyacht 2y agoYes, 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...