gnodet opened a new pull request, #21906:
URL: https://github.com/apache/camel/pull/21906
## Summary
- React with :+1: on the trigger comment instead of posting a separate
"robot will run tests" comment — lightweight, non-noisy acknowledgment
- Post test results (success or failure) as **new comments** on the PR
instead of editing the original `/component-test` comment, which was confusing
as it replaced the user's text
- Add workflow-level failure comment for early failures (sha check,
checkout, Java setup) that occur before the component-test action runs
- Fix checkout ref: `${{ env.pr_sha }}` → `${{ steps.pr.outputs.pr_sha }}`
(was never set as an env var — latent bug)
- Fix hardcoded `apache/camel` in failure URL to use `${{ github.repository
}}`
## Behavior changes
| Before | After |
|---|---|
| Posts "🤖 The Apache Camel test robot will run the tests for you 👍" comment
| Adds 👍 reaction to the `/component-test` comment |
| Edits the original `/component-test` comment to append results | Posts a
new comment with results |
| No feedback on early failures (sha check, setup) | Posts failure comment
with link to logs |
## Test plan
- [ ] Post `/component-test camel-jms` on a PR and verify 👍 reaction appears
on the comment
- [ ] Verify test results are posted as a new comment (not editing the
original)
- [ ] Verify failure scenario posts a comment with log link
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
