Skip to content

Poll GitHub Actions results from forked repositories as PR checks#164

Merged
raeessap merged 4 commits into
sapfrom
forked_prs
Jul 2, 2026
Merged

Poll GitHub Actions results from forked repositories as PR checks#164
raeessap merged 4 commits into
sapfrom
forked_prs

Conversation

@raeessap

@raeessap raeessap commented Jul 2, 2026

Copy link
Copy Markdown

the github checks will be performed on the user's forked repository and if a PR is created from a forked repo, the upstream github will read the results of the checks that ran on forked repo.

TGSAPJVM-6151

@raeessap raeessap requested a review from RealCLanger July 2, 2026 08:54

@RealCLanger RealCLanger left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One comment about not modifying validate.yml. And I assume the sync_fork-checks just works. 😄 When you then merge it, remove the sap: from the commit message, it is not needed or not a convention.

Comment thread .github/workflows/validate.yml Outdated
@raeessap raeessap changed the title sap: Poll GitHub Actions results from forked repositories as PR checks Poll GitHub Actions results from forked repositories as PR checks Jul 2, 2026
@raeessap

raeessap commented Jul 2, 2026

Copy link
Copy Markdown
Author

One comment about not modifying validate.yml. And I assume the sync_fork-checks just works. 😄 When you then merge it, remove the sap: from the commit message, it is not needed or not a convention.

i can only perform tests after merging. :)

@raeessap raeessap merged commit 1dda319 into sap Jul 2, 2026
4 checks passed
@raeessap raeessap deleted the forked_prs branch July 2, 2026 09:11
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.

2 participants