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.

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

Remove duplicate Github actions for every PR #166

Merged
merged 2 commits into from
Nov 16, 2023

Conversation

prithvi514
Copy link
Contributor

Updating Github CI actions to trigger on pushing to master and for pull requests that target master branch.

@prithvi514 prithvi514 requested a review from a team as a code owner November 15, 2023 22:37
@prithvi514 prithvi514 requested review from amortezaei and petedannemann and removed request for a team November 15, 2023 22:39
push:
branches:
- master
pull_request:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This means people cannot target other branches in pull requests. I have been doing this recently as I have many branches that stack upon each other and this has made it easier to chunk up large features into several smaller PRs. Is there a way around this limitation?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if something like this is what we want:

push:
    branches:
      - master
  pull_request:

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yea, this sounds good.

@prithvi514 prithvi514 merged commit d5c5b3a into master Nov 16, 2023
6 checks passed
@prithvi514 prithvi514 deleted the dp-1746/duplicate-ci-steps branch November 16, 2023 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants