Skip to content

fix: regenerate rules/index.json with @commits trigger - #109

Merged
bborbe merged 1 commit into
masterfrom
fix/stale-rules-index
Aug 17, 2026
Merged

fix: regenerate rules/index.json with @commits trigger#109
bborbe merged 1 commit into
masterfrom
fix/stale-rules-index

Conversation

@bborbe

Copy link
Copy Markdown
Owner

Regenerate rules/index.json via make build-index. The committed index was missing the trigger: ["@commits"] entry for git-workflow/never-direct-commit-to-master (the **Trigger**: @commits line is in docs since 7a8c11d), so make precommit -> check-index failed on every push. make check-index now passes.

@ben-s-pull-request-reviewerben-s-pull-request-reviewerBot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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


PR Review Report

No changes to review — the diff is clean and correct.

Verdict: approve

This PR adds the missing trigger: ["@commits"] field to git-workflow/never-direct-commit-to-master in rules/index.json, which is exactly the right fix. The CHANGELOG entry uses conventional commit format. The JSON is valid. No issues found.


{
"verdict": "approve",
"summary": "Clean bugfix: adds the missing @commits trigger to git-workflow/never-direct-commit-to-master so make precommit (check-index) no longer fails. CHANGELOG entry follows conventions. No issues found.",
"comments": [],
"concerns_addressed": []
}

@bborbe
bborbe merged commit f199a8a into masterAug 17, 2026
1 check passed
@bborbe
bborbe deleted the fix/stale-rules-index branch August 17, 2026 21:02
Sign up for freeto 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.

1 participant

@bborbe