Allow init --skip-env and --only-env #2371
Open
+61
−20
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
In (some?) shells, it's sometimes preferable to set up env vars in a
different place than other configuration like most of what
atuin init
sets up. For example, ZSH reads ~/.zshenv for all shells while ~/.zshrc
is only read for interactive shells.
This allows splitting the output of init such that in these two
different config files we could load environment variables and load all
other config seperately:
~/.zshenv:
~/.zshrc:
Which means that when launching noninteractive shells (I do this
frequently for running tests from Vim, for example, but it's more
generally useful) the specified env values will be sourced and we won't
redefine them in login shells.
Checks
to speed up the review cycle
Note: I know I didn't create an issue or post anywhere else first. If
this is totally not a desired feature my feelings won't be hurt if you
say so and close this.