Skip to content

fix: skip fork Project sync events - #10909

Merged
zouyonghe merged 1 commit into
AstrBotDevs:masterfrom
zouyonghe:fix/project-sync-skip-fork-pr
Oct 31, 2026
Merged

zouyonghe merged 1 commit into
AstrBotDevs:masterfrom
zouyonghe:fix/project-sync-skip-fork-pr

Conversation

@zouyonghe

@zouyonghe zouyonghe commented Oct 31, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Skip jobs when the PR head repository is a fork.
  • Preserve same-repository PR sync; scheduled full reconciliation still catches fork PR final state.
  • Document the behavior and add a workflow-condition regression test.

Verification

  • 15 focused entrypoint/workflow tests pass.
  • Full-repository Ruff and Actionlint pass.

Follow-up to #10905; no issue, token, or Project behavior changed.

Summary by Sourcery

Prevent unnecessary Project synchronization runs for fork-origin pull request events while retaining same-repository synchronization and scheduled reconciliation.

Bug Fixes:

  • Skip Project synchronization jobs for pull request events originating from fork repositories while preserving synchronization for same-repository pull requests.

Enhancements:

  • Document that fork pull request state is handled by scheduled or manual reconciliation.

Documentation:

  • Document the handling of fork-origin pull request events and scheduled reconciliation.

Tests:

  • Add a regression test verifying fork pull request events are filtered at the workflow job boundary.

@sourcery-ai sourcery-ai Bot 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.

Hey - I've reviewed your changes and they look great!

Sourcery assessment

Approved.


Sourcery is free for open source - if you like our reviews please consider sharing them ✨

@zouyonghe
zouyonghe merged commit 831abae into AstrBotDevs:master Oct 31, 2026
26 checks passed
@Soulter Soulter moved this to Done in AstrBot Roadmap Nov 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant