-
Notifications
You must be signed in to change notification settings - Fork 139
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Feature request] Environment variable for non-standard install location #562
Comments
Also:
|
This partially overlaps with #559, so I'll need to have that other PR merged first before I can start looking into this feature request 🤷🏻 |
I'd actually like to see the -i option added to the TOML file as well. Had a situation come up at work the other day where this would come in very handy. On a shared server with a small C drive, if we have a bunch of users that work with multiple different terraform versions it can start to add up. Having to pass -i to tfswitch all the time is a pain so being able to default the alternate location in the TOML file would be great. |
@mj790 Thanks for describing your use case. Usual "thumbs up" to the first message would allow us to easily prioritize issues 👍🏻 I'll work on this request once that other PR is reviewed and approved by @warrensbox and @MatthewJohn. Stay tuned 🎶 |
Resolves #562 * Add Env vars for custom binary path and custom install path * Add TOML config key for custom install path for consistency * Improve wording in resolved parameter values debugging output * Add tests for new Env vars and TOML config key * Documentation * Update https://tfswitch.warrensbox.com/usage/commandline/ accordingly * Update https://tfswitch.warrensbox.com/usage/config-files/ accordingly * Add `arch` and `log-level` TOML keys description to https://tfswitch.warrensbox.com/usage/config-files/ * Improve wording in some places
#566 |
Discussed in #561
Originally posted by dbopus2 March 10, 2025
Would you consider adding an environment variable for non-standard install location? Same functionality as the -i and --install cli options.
The text was updated successfully, but these errors were encountered: