Skip to content

Allow co-author checks on upstream sync PRs - #640

Open
sharath58 wants to merge 1 commit into
yc-software:mainfrom
sharath58:fix-coauthor-sync-range
Open

Allow co-author checks on upstream sync PRs#640
sharath58 wants to merge 1 commit into
yc-software:mainfrom
sharath58:fix-coauthor-sync-range

Conversation

@sharath58

@sharath58 sharath58 commented Aug 20, 2026

Copy link
Copy Markdown

Summary

  • Exclude commits already reachable from official qm main when validating co-author trailers.
  • Continue rejecting GitHub privacy addresses introduced by the pull request itself.

Why

Merge-based upstream syncs place historical upstream commits in the pull request range. A trailer policy added later should not retroactively reject commits that official qm already accepted.

Verification

  • Prettier check for the workflow.
  • Replayed the check against a merge-based sync range containing historical privacy-address trailers.
  • Scrubbed outgoing history for organization paths, identifiers, and binaries.

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

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.

1 participant