Skip to content

ci: bump contributor-governance pin to include #7 fix - #1373

Merged
david-martin merged 1 commit into
mainfrom
fix/bump-contributor-governance-pin
Aug 14, 2026
Merged

ci: bump contributor-governance pin to include #7 fix#1373
david-martin merged 1 commit into
mainfrom
fix/bump-contributor-governance-pin

Conversation

@david-martin

@david-martin david-martin commented Aug 14, 2026

Copy link
Copy Markdown
Member

The contributor-governance workflow pins the reusable workflow to a frozen SHA (dfca825, from Kuadrant/.github#6) that predates Kuadrant/.github#7. #7 removed the "PR author must be assigned to the linked issue" check, but this repo never re-ratcheted the pin, so it still runs the old logic and auto-closes external-contributor PRs (e.g. #1370) on reopen.

Bumps the pin dfca825 -> d161768 (current Kuadrant/.github main HEAD, includes #7).

Since pull_request_target uses the workflow from the base branch, this only takes effect once merged. After merge, #1370 can be reopened without re-closing.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated the contributor governance workflow to use a newer version of the shared governance process.

The pinned SHA (dfca825) predated Kuadrant/.github#7, which removed the
'PR author must be assigned to the linked issue' check. Bump to current
main HEAD (d161768) so external-contributor PRs are no longer auto-closed
on that condition.

Signed-off-by: David Martin <davmarti@redhat.com>
@coderabbitai

coderabbitai Bot commented Aug 14, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 1e71b6c5-08b9-4081-9a5c-ea489d2c999d

📥 Commits

Reviewing files that changed from the base of the PR and between 29396cf and 76c459d.

📒 Files selected for processing (1)
  • .github/workflows/contributor-governance.yml

📝 Walkthrough

Walkthrough

The contributor governance workflow now references a newer pinned revision of the shared governance workflow.

Changes

Contributor governance

Layer / File(s) Summary
Update governance workflow reference
.github/workflows/contributor-governance.yml
The workflow uses a different pinned commit for the shared contributor governance workflow.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 76c45

This localized workflow update advances the contributor-governance pin to include the upstream fix, preventing the outdated author-assignment check from continuing to affect external-contributor pull requests. No actionable merge-blocking risk remains beyond normal checks and review.

Suggested labels: review-effort/small

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the contributor-governance workflow pin update and its purpose.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/bump-contributor-governance-pin

Warning

Review ran into problems

🔥 Problems

Git: Failed to clone repository. Please run the @coderabbitai full review command to re-trigger a full review. If the issue persists, set path_filters to include or exclude specific files.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot added the review-effort/small Low review effort (1-2): straightforward, single file, config/docs label Aug 14, 2026
@david-martin
david-martin merged commit e3e886c into main Aug 14, 2026
18 of 21 checks passed
christinaexyou pushed a commit to christinaexyou/mcp-gateway that referenced this pull request Aug 17, 2026
…nt#1373)

The pinned SHA (dfca825) predated Kuadrant/.github#7, which removed the
'PR author must be assigned to the linked issue' check. Bump to current
main HEAD (d161768) so external-contributor PRs are no longer auto-closed
on that condition.

Signed-off-by: David Martin <davmarti@redhat.com>
kkukadia pushed a commit to kkukadia/mcp-gateway that referenced this pull request Aug 18, 2026
…nt#1373)

The pinned SHA (dfca825) predated Kuadrant/.github#7, which removed the
'PR author must be assigned to the linked issue' check. Bump to current
main HEAD (d161768) so external-contributor PRs are no longer auto-closed
on that condition.

Signed-off-by: David Martin <davmarti@redhat.com>
christinaexyou pushed a commit to christinaexyou/mcp-gateway that referenced this pull request Aug 20, 2026
…nt#1373)

The pinned SHA (dfca825) predated Kuadrant/.github#7, which removed the
'PR author must be assigned to the linked issue' check. Bump to current
main HEAD (d161768) so external-contributor PRs are no longer auto-closed
on that condition.

Signed-off-by: David Martin <davmarti@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review-effort/small Low review effort (1-2): straightforward, single file, config/docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants