Skip to content
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

feat: add support for env defined binary #68

Merged
merged 1 commit into from
Feb 17, 2024
Merged

feat: add support for env defined binary #68

merged 1 commit into from
Feb 17, 2024

Conversation

quotidian-ennui
Copy link
Contributor

Resolves #67

Changes

Resolves a TF_BINARY environment variable that tells us what command to execute. Still assumes it's on the path so no behavioural difference.
Defaults to terraform for backwards compatibility.

Notes

  • Choice of TF_BINARY was pretty arbitrary.
  • I'm not primarily a go dev and there didn't seem to be an standardised way of get_env_or_default equivalent.
  • I tested locally by executing the output from make build manually (so no additional tests).

@dineshba dineshba merged commit 2c17fcc into dineshba:main Feb 17, 2024
1 check passed
@quotidian-ennui quotidian-ennui deleted the feat/env-defined-terraform branch February 17, 2024 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add support for opentofu for binary plans
2 participants