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

Sentinel Fmt Hook Does Not Work #121

Open
2 tasks
schultzwc18 opened this issue Feb 1, 2025 · 0 comments
Open
2 tasks

Sentinel Fmt Hook Does Not Work #121

schultzwc18 opened this issue Feb 1, 2025 · 0 comments
Labels
bug Something isn't working

Comments

@schultzwc18
Copy link

Describe the bug
Hook fails to run when installed through pre-commit because of two bugs:

  1. Line 1 must have shebang but is instead on line 2
  2. sentinel fmt does not have a diff command-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.

@schultzwc18 schultzwc18 added the bug Something isn't working label Feb 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant