Please add an option to download/install terraform
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 35/100
Research direction
The failure points to hooks/terraform-fmt.sh; start there and review how this hook is invoked on pre-commit.ci. Add the requested Terraform installation option and verify that terraform-fmt runs successfully when Terraform is absent, while preserving current behavior when it is already installed.
Written by the indexing model from the issue text.
Description
Describe the solution you'd like
I would like to be able to use terraform-fmt on pre-commit.ci, the hosted CI pre-commit service, but unfortunately I cannot because terraform doesn't come pre-installed, and the hooks here do not install/bundle it.
Terraform fmt............................................................Failed
- hook id: terraform-fmt
- exit code: 127
/pc/clone/RKgVfIZxQCiSAX665qKF9Q/hooks/terraform-fmt.sh: line 15: terraform: command not found
It'd be nice if there was a way to say that you need terraform installed as part of the hook.
Describe alternatives you've considered
Not running terraform hooks on pre-commit.ci.
- Dominant language
- Shell
- Stars
- 546
- Forks
- 167
- PR merge metrics
- No merged PRs in 30d
Getting set up
- No Dockerfile or Docker Compose file
- Has a pull request template
- No contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from gruntwork-io/pre-commit
-
Documentation for shellcheck should include that it is configurable with a .shellcheckrcMay be free again A pull request for this issue was closed without being merged. Openenhancement
Difficulty 1/5 Under an hour Newbie friendliness 68/100
gruntwork-io/pre-commit#79 ·
-
Support args for terraform-fmt, remove hardcoded args.May be free again @ZachGoldberg claimed this 385 days ago, and no pull request is open. Openenhancement
gruntwork-io/pre-commit#138 · 3 reactions · 1 assignee ·
-
Sentinel Fmt Hook Does Not WorkMay be free again A pull request for this issue was closed without being merged. Openbug
Difficulty 2/5 1-3 hours Newbie friendliness 38/100
gruntwork-io/pre-commit#121 ·
-
enhancement
Difficulty 3/5 1-2 days Newbie friendliness 35/100
gruntwork-io/pre-commit#111 · 1 reaction ·
-
bug
Difficulty 1/5 Under an hour Newbie friendliness 45/100
gruntwork-io/pre-commit#110 ·
All issues in gruntwork-io/pre-commit
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 1 day
-
area/tests theme/ci-dx
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
Maintainers usually reply within 1 day
-
[Bug]: e2e script flag parsing is brokenPossibly taken @ericcurtin claimed this today. Openarea/dev-infra area/tests kind/bug
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
agent-substrate/substrate#2217 · 1 comment ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
FluidNumerics/fluid-walk-blocker#172 ·
Maintainers usually reply within 1 day