Skip to content

Gh pr pre commit check#13

Open
shubhamdp wants to merge 3 commits intomainfrom
gh-pr-pre-commit-check
Open

Gh pr pre commit check#13
shubhamdp wants to merge 3 commits intomainfrom
gh-pr-pre-commit-check

Conversation

@shubhamdp
Copy link
Owner

@shubhamdp shubhamdp commented Feb 20, 2026

Note

Low Risk
CI/workflow and developer tooling changes only; main risk is new PR check failures or CI noise if hook behavior (especially codespell --write-changes) is unexpected.

Overview
Adds a new GitHub Actions workflow (.github/workflows/pre-commit.yml) that runs pre-commit on pull requests to main and release/**, scoped to the PR diff via --from-ref/--to-ref.

Expands .pre-commit-config.yaml beyond astyle_py to also enforce sorted blocks (keep-sorted) and run spell checking with auto-fixes (codespell --write-changes), and adds DEVELOPER_GUIDE.md documenting how to install and run these hooks locally.

Written by Cursor Bugbot for commit 4b73bb0. This will update automatically on new commits. Configure here.

shubhamdp and others added 2 commits February 20, 2026 14:10
- keep-sorted: for sorting the lists between start/stop marker
- codespell: to fix the spelling mistakes
git-hook: added keep-sorted and codespell pre-commit hooks

See merge request app-frameworks/esp-matter!1354
@shubhamdp shubhamdp force-pushed the gh-pr-pre-commit-check branch from 316ad0a to 353e582 Compare February 20, 2026 13:22
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

@shubhamdp shubhamdp force-pushed the gh-pr-pre-commit-check branch from 353e582 to 4b73bb0 Compare February 20, 2026 15:24
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