-
Notifications
You must be signed in to change notification settings - Fork 135
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
Default version flag added #275
Default version flag added #275
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please outline what's the difference wuth TF_VERSION
env var (
Lines 209 to 213 in e77b48c
/* if Terraform Version environment variable is set */ | |
case checkTFEnvExist() && len(args) == 0: | |
tfversion := os.Getenv("TF_VERSION") | |
fmt.Printf("Terraform version environment variable: %s\n", tfversion) | |
installVersion(tfversion, custBinPath, mirrorURL) |
TF_VERSION
cannot suffice in your use case?Thanks.
Co-authored-by: George L. Yermulnik <yz@yz.kiev.ua>
@yermulnik In cases where TF_VERSION is already set (say for eg |
@sivaramsajeev Aha, got you. The new option is intended to be a fallback. |
The
|
Aah, I see now. This is the same as requested in #150 and #165 and actually is, broadly speaking, a programmatic implementation of "hackaround" from #165 (comment) (or |
I am still reviewing this |
This flag will allow us to specify a default version to install which will be useful for the CI systems instead of going into the prompt