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

#21 add support for passing terraform version to docker build #22

Merged
merged 1 commit into from
Jun 3, 2024
Merged

#21 add support for passing terraform version to docker build #22

merged 1 commit into from
Jun 3, 2024

Conversation

buker
Copy link
Contributor

@buker buker commented Jun 3, 2024

#21

Copy link
Collaborator

@maruloop maruloop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your PR!
It seems to need to use either ENV INPUT_TERRAFORM_VERSION or ARG with arg input.

https://docs.github.com/en/actions/creating-actions/metadata-syntax-for-github-actions#inputs

Both REVIEWDOG_VERSION and TERRAFORM_VERSION will be variables needed only at build time, so ARG is more appropriate than ENV.

@maruloop maruloop merged commit 0b7cb3c into reviewdog:main Jun 3, 2024
8 checks passed
@review-dog
Copy link
Member

Hi, @buker! We merged your PR to reviewdog! 🐶
Thank you for your contribution! ✨

We just invited you to join the @reviewdog organization on GitHub.
Accept the invite by visiting https://github.com/orgs/reviewdog/invitation.
By joining the team, you'll be a part of reviewdog community and can help the maintenance of reviewdog.

Thanks again!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants