Skip to content

Remove retired PR Governance gate - #1665

Merged
groupthinking merged 2 commits into
mainfrom
copilot/remove-pr-governance-gate
Sep 8, 2026
Merged

Remove retired PR Governance gate#1665
groupthinking merged 2 commits into
mainfrom
copilot/remove-pr-governance-gate

Conversation

Copilot AI commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Canonical issue

Issue linking is handled by the PR metadata.

Outcome

Retires the unwanted PR Governance gate so PRs and Dependabot auto-merge no longer wait on checks that will never report.

Scope

  • Included:
    • Deleted pr-governance.yml and its workflow unit test.
    • Removed PR Governance / Canonical issue and evidence from Dependabot required checks.
    • Updated merge policy and workflow docs to treat binding as policy-level only.
  • Explicitly excluded:
    • PR template changes.
    • Historical runbook edits.

Risk

  • Risk level: low
  • Failure mode: a stale reference could keep another gate waiting on removed checks.
  • Rollback: restore the deleted workflow/test and re-add both check names to the Dependabot gate and policy docs.

Verification

  • Focused tests: pytest tests/unit/test_dependabot_automation_workflow.py -q --no-cov
  • Required CI
  • Review threads resolved

Additional checks:

  • Reference grep leaves only the allowed historical runbook hit.
  • All remaining .github/workflows/*.yml parse with PyYAML.

Production evidence

Not applicable. This removes a GitHub Actions governance workflow and stale references; no runtime deployment path changes.

Agent handoff

  • One canonical issue is linked
  • No competing PR implements the same issue
  • Acceptance criteria are satisfied
  • Required checks pass on the current head
  • Human decision is requested only for product, security, irreversible infrastructure, or production approval

@vercel

vercel Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
v0-uvai Ready Ready Preview, v0 Sep 8, 2026 4:50am UTC

Co-authored-by: groupthinking <154503486+groupthinking@users.noreply.github.com>
installation tokens; the workflow now calls the REST assignees endpoint.


## Agent completion enforcement

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.

Duplicated ## Agent completion enforcement and ## Repository governance workflows headings garbled the markdown in AUDIT.md

Fix on Vercel

Copilot AI changed the title [WIP] Remove PR Governance GitHub Actions workflow Remove retired PR Governance gate Sep 8, 2026
Copilot AI requested a review from groupthinking September 8, 2026 04:53
@groupthinking
groupthinking marked this pull request as ready for review September 8, 2026 04:59
Copilot AI balanced review requested due to automatic review settings September 8, 2026 04:59
@github-actions github-actions Bot added ci/cd documentation Improvements or additions to documentation python labels Sep 8, 2026
@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown

🔍 PR Validation

⚠️ PR title should follow conventional commits format
⚠️ Large PR detected (545 lines changed)

@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 41fe393.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

  • .github/workflows/pr-governance.yml

Copilot AI 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.

🟡 Changes recommended

The audit document introduces two pairs of duplicate section headings.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Retires the PR Governance workflow and synchronizes Dependabot automation, tests, and policy documentation.

Changes:

  • Deletes the governance workflow and its tests.
  • Removes retired checks from Dependabot’s merge gate.
  • Documents issue binding as policy-only.
File summaries
File Description
.github/workflows/pr-governance.yml Removes the retired workflow.
tests/unit/test_pr_governance_workflow.py Removes obsolete tests.
.github/workflows/dependabot-auto-merge.yml Drops retired required checks.
tests/unit/test_dependabot_automation_workflow.py Updates merge-gate expectations.
MERGE_POLICY.md Makes binding policy-only.
.jules/agent_orchestration_sop.md Removes obsolete gate guidance.
.github/workflows/README.md Updates workflow documentation.
.github/workflows/AUDIT.md Updates governance audit history.
Review details

Suppressed comments (1)

.github/workflows/AUDIT.md:70

  • This duplicates the existing Repository governance workflows heading on the next line, creating another empty section in the rendered audit. Keep only one heading.
## Repository governance workflows
  • Files reviewed: 8/8 changed files
  • Comments generated: 1
  • Review effort level: Balanced

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment on lines +65 to 66
## Agent completion enforcement
## Agent completion enforcement
@groupthinking
groupthinking merged commit b46b392 into main Sep 8, 2026
31 of 36 checks passed
@groupthinking
groupthinking deleted the copilot/remove-pr-governance-gate branch September 8, 2026 05:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/cd documentation Improvements or additions to documentation python

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants