-
Notifications
You must be signed in to change notification settings - Fork 159
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
fix workflows for fork PRs #3728
Conversation
605dcca
to
e791837
Compare
e791837
to
f9cb36c
Compare
@@ -15,6 +15,10 @@ on: | |||
# This needs to be declared explicitly so that the job is actually | |||
# run when moved out of draft. | |||
types: [opened, synchronize, reopened, ready_for_review] | |||
pull_request_target: |
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.
Does this one not need branches?
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.
For consistency it should, I'll address it in another PR given it's outside of the scope of this one, and I'd like to have an easy way of reverting this if needed.
This reverts commit b7ac403.
Summary of changes
Changes introduced in this pull request:
Reference issue to close (if applicable)
Closes
Other information and links
Change checklist