ci: report test status to changes made from a forked branch #36
Conversation
zimeg
added
semver:patch
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #36 +/- ##
==========================================
+ Coverage 62.92% 62.93% +0.01%
==========================================
Files 210 210
Lines 22149 22149
==========================================
+ Hits 13937 13940 +3
+ Misses 7126 7125 -1
+ Partials 1086 1084 -2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
…nch names this workflow file is checked out with the pull request changes so this wasnt updating things as hoped
zimeg
left a comment
zimeg
left a comment
There was a problem hiding this comment.
📝 A note of instruction for reviewers!
One more note is that the status won't appear when "main" is the selected workflow as these changes are not on main. For now this branch can be selected instead when testing 🦠
| - End-to-end tests for pull requests from forks can be [started][e2e] if: | ||
| - The `workflow` from the **main** branch is used | ||
| - The `branch` contains the pull request number as: `pull/123/head` | ||
| - The `status` is reported with the commit checks |
There was a problem hiding this comment.
📚 In rendered markdown this can be found at this branch link:
|
@mwbrooks I am hoping so too 🙏 ✨ And thanks for the quick review! I'll hold off on merging right away since pre-releases remain when branches of a forked PR are deleted and of course for perhaps additional comments and more testing as needed 👾 |
This is proving to be an unrelated change AFAICT! I'll instead merge this now and follow up with the changes needed. Please do stay tuned 📻 |
Summary
This PR adds an option to report the status of E2E tests to PRs made from a forked branch.
Requirements