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

Added custom pre-commit hook #2499

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

hrishi-1337
Copy link

@hrishi-1337 hrishi-1337 commented Oct 11, 2024

Custom pre-commit hook to ensure indentation consistency of JSON files. Resolves #2332

Checklist

  • Type annotations added to new functions
  • Docs added to functions touched in main classes
  • Dry-run produced the expected results
  • The to-be-announced tag added if this is worth announcing
$ pre-commit run --all-files
djLint linting for Jinja.................................................Passed
djLint linting for Jinja.................................................Passed
ruff.....................................................................Passed
ruff-format..............................................................Passed
check python ast.........................................................Passed
check docstring is first.................................................Passed
check that executables have shebangs.....................................Passed
check for merge conflicts................................................Passed
check for broken symlinks............................(no files to check)Skipped
debug statements (python)................................................Passed
trim trailing whitespace.................................................Passed
check yaml...............................................................Passed
mixed line ending........................................................Passed
python tests naming......................................................Passed
check json...............................................................Passed
fix requirements.txt.....................................................Passed
check vcs permalinks.....................................................Passed
codespell................................................................Passed
taskcluster_yml..........................................................Passed
Strip unnecessary `# noqa`s..............................................Passed
mypy.....................................................................Passed
Check for useless excludes...............................................Passed
Check JSON Indentation Consistency.......................................Passed


@suhaibmujahid suhaibmujahid self-assigned this Oct 11, 2024
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.

pre-commit allows inconsistent mix between tab and space indentation in JSON files
2 participants