Sentinel Fmt Hook Does Not Work
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 38/100
Research direction
Start by reproducing the issue with the provided .pre-commit-config.yaml and the pre-commit run --all-files command. Inspect the sentinel-fmt hook definition and verify that its shebang is on line 1 and that the invoked sentinel fmt command supports the arguments used; done means the hook runs and reports files requiring formatting.
Written by the indexing model from the issue text.
Description
Describe the bug
Hook fails to run when installed through pre-commit because of two bugs:
- Line 1 must have shebang but is instead on line 2
sentinel fmtdoes not have adiffcommand-line argument
To Reproduce
Steps to reproduce the behavior including the relevant Terraform/Terragrunt/Packer version number and any code snippets and module inputs you used.
.pre-commit-config.yaml
repos:
- repo: https://github.com/gruntwork-io/pre-commit
rev: v0.1.25
hooks:
- id: sentinel-fmt
pre-commit install
pre-commit run --all-files
Expected behavior
Pre-commit hook would run and indicate files requiring formatting
Nice to have
- Terminal output
- Screenshots
Additional context
Add any other context about the problem here.
- Dominant language
- Shell
- Stars
- 547
- Forks
- 168
- PR merge metrics
- No merged PRs in 30d
Contributor guide
No contributing guide indexed for this repository
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
-
enhancement
Difficulty 1/5 Under an hour Newbie friendliness 68/100
gruntwork-io/pre-commit#79 ·
-
enhancement
gruntwork-io/pre-commit#138 · 3 reactions · 1 assignee ·
-
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 ·
-
OpenTofu Support Openenhancement
Difficulty 2/5 1-3 hours Newbie friendliness 25/100
gruntwork-io/pre-commit#108 · 1 comment · 2 reactions ·
All issues in gruntwork-io/pre-commit
Similar issues
-
Difficulty 1/5 Under an hour Newbie friendliness 92/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
community-scripts/ProxmoxVE#17425 · 1 comment ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 90/100
danielmiessler/LifeOS#2218 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
conda-forge/python-feedstock#934 ·
-
docs(agents): strengthen the no-backslash-escaped-backticks rule with an issue-creation example Open
Difficulty 1/5 Under an hour Newbie friendliness 92/100