Skip to content

ci: use workflow_run to publish test results on fork PRs#2450

Merged
jbonofre merged 1 commit intoapache:mainfrom
jbonofre:ci-test-results-workflow
Mar 18, 2026
Merged

ci: use workflow_run to publish test results on fork PRs#2450
jbonofre merged 1 commit intoapache:mainfrom
jbonofre:ci-test-results-workflow

Conversation

@jbonofre
Copy link
Copy Markdown
Member

The publish-unit-test-result-action requires write permissions to post PR comments and check runs, but fork PRs run with read-only tokens. Move test result publishing to a separate workflow triggered by workflow_run, which runs in the base repo context with write access.

The publish-unit-test-result-action requires write permissions to post
PR comments and check runs, but fork PRs run with read-only tokens.
Move test result publishing to a separate workflow triggered by
workflow_run, which runs in the base repo context with write access.
jbonofre added a commit to jbonofre/karaf that referenced this pull request Mar 18, 2026
…ache#2450)

The publish-unit-test-result-action requires write permissions to post
PR comments and check runs, but fork PRs run with read-only tokens.
Move test result publishing to a separate workflow triggered by
workflow_run, which runs in the base repo context with write access.
@jbonofre jbonofre merged commit e3d9d76 into apache:main Mar 18, 2026
2 checks passed
@jbonofre jbonofre deleted the ci-test-results-workflow branch March 18, 2026 04:27
jbonofre added a commit that referenced this pull request Mar 18, 2026
) (#2458)

The publish-unit-test-result-action requires write permissions to post
PR comments and check runs, but fork PRs run with read-only tokens.
Move test result publishing to a separate workflow triggered by
workflow_run, which runs in the base repo context with write access.
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