Skip to content

[COVAL-7325] fix: allow parity guard comments on pull requests - #143

Merged
callumreid merged 1 commit into
mainfrom
fix/coval-7325-parity-comment-permission
Nov 2, 2026
Merged

callumreid merged 1 commit into
mainfrom
fix/coval-7325-parity-comment-permission

Conversation

@callumreid

Copy link
Copy Markdown
Contributor
  • Give the weekly parity job pull-requests: write so it can comment on an existing hand-reconciled rolling PR. Retain read-only repository contents and the existing issue-reporting permission.
  • The November 2 run preserved [COVAL-5819] feat: add Python metric draft commands and audit request fields #141 correctly, then failed with HTTP 403 when posting its guard comment: https://github.com/coval-ai/cli/actions/runs/41425914844. No branch reset, credential change, or broader contents access is introduced.
  • Validation: actionlint and all 63 existing automation tests pass. Actual scheduled-workflow recovery remains to be verified after this change merges; no generator rerun or hand-branch mutation was used for validation.

@borgesius borgesius left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good. This supplies the pull-request comment permission missing in the recorded failure while preserving read-only contents access and the hand-commit branch guard.

@callumreid
callumreid merged commit b7dadf6 into main Nov 2, 2026
10 checks passed
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