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

Which terraform #535

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Which terraform #535

wants to merge 3 commits into from

Conversation

jcollado
Copy link

@jcollado jcollado commented Oct 16, 2024

Description

Check that terraform binary is available before trying to use it.

Motivation and Context

Remove error messages when a .terraform directory is present, but the terraform binary is not available.

Closes #534

Screenshots (if appropriate)

How Has This Been Tested

I have used the code in this with/without terraform being present, and it worked as expected.

  • I have tested using Linux.
  • I have tested using MacOS.

Checklist

  • I am ready to update the wiki accordingly.
  • I have updated the tests accordingly.

@jcollado jcollado requested a review from IlanCosman as a code owner October 16, 2024 13:29
@jcollado jcollado force-pushed the which-terraform branch 3 times, most recently from d6b703c to 9f42bc6 Compare October 16, 2024 14:06
@jcollado
Copy link
Author

A couple of comments regarding the checklist:

  • I believe that the wiki doesn't need to be updated.
  • I've updated the test cases so that they pass. However, I don't know what would be the right way to add a test to verify the situation when the terraform command is not found.

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.

Unknown command terraform
1 participant