Skip to content

Add PR artifact uploads and comments#11

Open
Stefterv wants to merge 6 commits intomainfrom
pr-fix
Open

Add PR artifact uploads and comments#11
Stefterv wants to merge 6 commits intomainfrom
pr-fix

Conversation

@Stefterv
Copy link
Owner

@Stefterv Stefterv commented Feb 2, 2026

Update the build workflow to handle pull request runs: make the main artifact upload conditional (skip for pull_request), add a separate upload step for PR artifacts with longer retention, and post a sticky PR comment containing a download link. Also adjust workflow permissions to allow pull-request write access and include updated artifact paths.

Update the build workflow to handle pull request runs: make the main artifact upload conditional (skip for pull_request), add a separate upload step for PR artifacts with longer retention, and post a sticky PR comment containing a download link. Also adjust workflow permissions to allow pull-request write access and include updated artifact paths.
@Stefterv Stefterv closed this Feb 2, 2026
@Stefterv Stefterv reopened this Feb 2, 2026
@github-actions
Copy link

github-actions bot commented Feb 2, 2026

(linux/aarch64) Build Artifacts

(macos/aarch64) Build Artifacts

(linux/x64) Build Artifacts

(windows/x64) Build Artifacts

(macos/x64) Build Artifacts

Add PR-facing comments and artifact wiring to the CI workflow. Post a sticky comment after Gradle tests to indicate artifacts availability, add an upload-artifact step with an id, and update the sticky-comment step to use the artifact output URL (steps.upload-artifact.outputs.artifact-url) and a consistent header. These changes make build artifacts discoverable on pull requests.
@github-actions
Copy link

github-actions bot commented Feb 2, 2026

Tests completed successfully. Build artifacts for this pull request will appear below once ready.

Build Artifacts

Platform Link
(macos/x64) Download processing-macos-x64-pr_11
(linux/aarch64) Download processing-linux-aarch64-pr_11
(linux/x64) Download processing-linux-x64-pr_11
(macos/aarch64) Download processing-macos-aarch64-pr_11
(windows/x64) Download processing-windows-x64-pr_11

@Stefterv
Copy link
Owner Author

Stefterv commented Feb 4, 2026

Tests completed successfully. Build artifacts for this pull request will appear below once ready.

Build Artifacts

Platform Link
(linux/aarch64) Download processing-linux-aarch64-pr_11
(linux/x64) Download processing-linux-x64-pr_11
(macos/aarch64) Download processing-macos-aarch64-pr_11
(macos/x64) Download processing-macos-x64-pr_11
(windows/x64) Download processing-windows-x64-pr_11

Comment out the steps in .github/workflows/build.yml that posted sticky pull-request comments with build artifact links (previously using marocchino/sticky-pull-request-comment and secrets.PR_PAT). Remove the separate .github/workflows/pr.yml workflow file. This disables automatic PR artifact comments and removes the now-unneeded duplicate PR workflow.
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