3 ms·
Be sure to put your env vars next to the invocation and not at the start, or it won't apply. Like for npm, correct way is "./bin/do.sh && NO_TELEMETRY=1 ven
by turtleyacht 2y ago
Be sure to put your env vars next to the invocation and not at the start, or it won't apply.
Like for npm, correct way is
"./bin/do.sh && NO_TELEMETRY=1 vendor build"
Incorrect:
"NO_TELEMETRY=1 ./bin/do.sh && vendor build"
With the latter, the environment variable to disable telemetry doesn't get passed in to the vendor script.
- 1oooqooq 2y agoyou 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...