Skip to content

Conversation

@sundargthb
Copy link
Contributor

Description

chore: add pre-commit hooks and fix fork PR testing

  • Configure pre-commit for formatting, linting, and tests
  • Enable integration tests for forked PRs with security checks
  • Update workflows to use pull_request_target safely

Improves developer experience and CI/CD pipeline by adding pre-commit hooks and enabling integration tests for forked PRs.

Changes

  • Pre-commit hooks: Enforce code formatting, linting, and tests before commit/push
  • Fork PR support: Integration tests now run on forked PRs with proper security controls
  • Security: Added safety checks to prevent workflow modification attacks

Testing

  • Pre-commit hooks tested locally
  • Fork PR workflow tested with test fork
  • All existing tests pass

Documentation

Updated CONTRIBUTING.md with pre-commit setup instructions.


Breaking Changes: None
Migration Required: Developers should run pre-commit install --install-hooks

Sundar Raghavan and others added 4 commits October 24, 2025 00:55
- Add pre-commit hooks for code quality
- Enable integration tests on forked PRs
- Add safety checks for workflow modifications
- Add pre-commit hooks for code quality
- Enable integration tests on forked PRs
- Add safety checks for workflow modifications
Removed an empty line at the beginning of the file.

Signed-off-by: Sundar Raghavan <[email protected]>
@sundargthb sundargthb requested a review from siwachabhi October 24, 2025 08:23
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.

2 participants