Skip to content
This repository was archived by the owner on Aug 14, 2025. It is now read-only.

chore(pre-commit): add a few more checks to pre-commit #23

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ashwinb
Copy link
Contributor

@ashwinb ashwinb commented Aug 13, 2025

TL;DR

Enhanced pre-commit configuration with additional hooks for improved code quality checks.

What changed?

  • Added --assume-in-merge argument to the check-merge-conflict hook
  • Added exclusion for Python files in trailing-whitespace hook to avoid duplication with Ruff
  • Added several new pre-commit hooks:
    • check-yaml with unsafe flag enabled
    • detect-private-key for security
    • check-executables-have-shebangs
    • check-json for JSON validation
    • check-shebang-scripts-are-executable
    • check-symlinks
    • check-toml for TOML validation

@meta-cla meta-cla bot added the cla signed label Aug 13, 2025
Copy link
Contributor Author

ashwinb commented Aug 13, 2025

@ashwinb ashwinb marked this pull request as ready for review August 13, 2025 03:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant