Skip to content

[Sync] Update project files from source repository (5ab2088) - #302

Merged
mrz1836 merged 1 commit into
masterfrom
chore/sync-files-go-mail-20261021-002503-5ab2088
Oct 21, 2026
Merged

mrz1836 merged 1 commit into
masterfrom
chore/sync-files-go-mail-20261021-002503-5ab2088

Conversation

@mrz1836

@mrz1836 mrz1836 commented Oct 21, 2026

Copy link
Copy Markdown
Owner

What Changed

  • GOVULNCHECK_GO_VERSION: 1.27.2 → 1.27.3 (.github/env/00-core.env)
  • The pull-request-management.yml workflow now also triggers on the synchronize event for same-repo PRs, so the "🔧 PR Management (Same Repo)" job runs on every new head commit.
  • On synchronize runs, the label and default-assignee steps are skipped, and the step summary reports "None: new commits only". Concurrency for these runs uses a separate -push group, so a push right after a PR opens cannot cancel the run that labels it.
  • Header comments and the "Trigger Configuration" notes were updated to explain the new behavior. Fork PR management through the workflow_run path still fires only on opened, reopened, ready_for_review and closed.

Why It Was Necessary

  • A check run belongs to a single commit. A ruleset that requires the same-repo PR management check would block any PR pushed to after it was opened, because the new head never received the check.
  • Running the job on synchronize without re-applying labels or the assignee reports the check on each new head. Maintainers can still override auto-applied labels without the workflow overwriting them on the next push.

Testing Performed

  • No automated tests are included in this sync. The change is limited to workflow configuration and an env value.
  • Validate by pushing a new commit to an open same-repo PR and confirming that the "🔧 PR Management (Same Repo)" check appears on the new head and that labels and assignee are unchanged.
  • Also confirm that opened, reopened, ready_for_review and closed events behave as before, and that fork PRs still go through the workflow_run path.

Impact / Risk

  • Risk: Low. The same-repo job now runs on each push, which adds a short run per push (10-minute timeout) but does no label or assignee work.
  • Concurrency: Newer pushes cancel older synchronize runs in the -push group, which is acceptable because the older commit is no longer the head. Runs for the other events keep their original group and are not affected.
  • Breaking changes: None expected. Required-check behavior for PRs pushed to after opening should improve.

@mrz1836 mrz1836 self-assigned this Oct 21, 2026
@mrz1836 mrz1836 added automated-sync Automated sync PR, e.g. from a fork or external repo automerge Label to automatically merge pull requests that meet all required conditions chore Simple dependency updates or version bumps labels Oct 21, 2026
@github-actions github-actions Bot added update General updates size/S Small change (11-50 lines) labels Oct 21, 2026
@mrz1836
mrz1836 merged commit 3957565 into master Oct 21, 2026
36 checks passed
@github-actions
github-actions Bot deleted the chore/sync-files-go-mail-20261021-002503-5ab2088 branch October 21, 2026 00:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automated-sync Automated sync PR, e.g. from a fork or external repo automerge Label to automatically merge pull requests that meet all required conditions chore Simple dependency updates or version bumps size/S Small change (11-50 lines) update General updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

⌂ 换网址