jetify-com / devbox

Instant, easy, and predictable development environments
https://www.jetify.com/devbox/
Apache License 2.0
7.83k stars 187 forks source link

[Devbox] introduce devopt.EnvOptions #2159

Closed savil closed 1 week ago

savil commented 2 weeks ago

Summary

From the EnvOptions docblock:

// EnvOptions configure the Devbox Environment in the `computeEnv` function.
// - These options are commonly set by flags in some Devbox commands
// like `shellenv`, `shell` and `run`.
// - The struct is designed for the "common case" to be zero-initialized as `EnvOptions{}`.

This gets rid of the pseudo-global state in the Devbox struct where we were setting pure, preservePathStack and omitNixEnv values.

How was it tested?

TODO:

sentry-io[bot] commented 5 days ago

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

Did you find this useful? React with a 👍 or 👎