-
Notifications
You must be signed in to change notification settings - Fork 277
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve fork experience #411
Conversation
Signed-off-by: Louise Poubel <[email protected]>
Codecov Report
@@ Coverage Diff @@
## ign-gazebo3 #411 +/- ##
============================================
Coverage 77.30% 77.30%
============================================
Files 205 205
Lines 11018 11018
============================================
Hits 8518 8518
Misses 2500 2500 Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@osrf-jenkins run tests please |
@osrf-jenkins run tests please |
1 similar comment
@osrf-jenkins run tests please |
Signed-off-by: Louise Poubel <[email protected]> Co-authored-by: Alejandro Hernández Cordero <[email protected]>
Signed-off-by: Louise Poubel <[email protected]> Co-authored-by: Alejandro Hernández Cordero <[email protected]> Signed-off-by: Guillaume Doisy <[email protected]>
Our GitHub actions haven't been running correctly for forks. This should fix all of them:
CI
CI only on
push
means that PRs from forks aren't tested. Addingpull_request
fixes forks, with the downside of triggering duplicated jobs for PRs coming from branches in this repo.Labeler
Resolves gazebo-tooling/pr-collection-labeler#4.
Uses the new pull_request_target event.
Tested here: gazebosim/testing#14
Triage
Like above, uses the new pull_request_target event.
Tested here: gazebosim/testing#15