Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Separate GH actions for repo branches and forks #479

Closed
pavish opened this issue Jul 23, 2021 · 0 comments · Fixed by #520
Closed

Separate GH actions for repo branches and forks #479

pavish opened this issue Jul 23, 2021 · 0 comments · Fixed by #520
Assignees
@kgodey
Labels
ready Ready for implementation work: other
Milestone
[04.1] 2021-08 im...

Comments

@pavish
Copy link
Member

pavish commented Jul 23, 2021

Description

Currently, we have GH actions on push, pull_request_target.
This runs the actions twice for every PR raised from a repo branch.

Expected behavior

When a PR is raised from a repo branch, the actions should only run once.
The actions should also run for PRs raised from forks.

To Reproduce

Raise a PR
@pavish pavish added type: bug work: other ready Ready for implementation labels Jul 23, 2021
@kgodey kgodey added the help wanted Community contributors can implement this label Jul 23, 2021
@kgodey kgodey added this to the 2021-09 improvements milestone Jul 23, 2021
@kgodey kgodey mentioned this issue Aug 3, 2021
Merged
7 tasks
@kgodey kgodey self-assigned this Aug 3, 2021
@kgodey kgodey removed the help wanted Community contributors can implement this label Aug 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kgodey kgodey

Labels
ready Ready for implementation work: other
Projects
No open projects
[NO LONGER USED] Mathesar work tracker
Archived in project
Milestone
[04.1] 2021-08 improvements
Development

Successfully merging a pull request may close this issue.

Added conditional for running tests on both push and PR mathesar-foundation/mathesar
2 participants
@kgodey @pavish