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

Run only workflow dispatch on default branch #2717

Merged
merged 3 commits into from
Dec 18, 2023

Conversation

Robert-Steiner
Copy link
Member

@Robert-Steiner Robert-Steiner commented Dec 18, 2023

Issue

Currently, it is possible to run the build server docker image CI job on branches other than the default branch (main).

Description

This PR adds a check that ensures that the CI job only runs on the default branch. If this is not the case, the CI job will be skipped.

Related issues/PRs

Proposal

Explanation

Checklist

  • Implement proposed change
  • Write tests
  • Update documentation
  • Update changelog
  • Make CI checks pass
  • Ping maintainers on Slack (channel #contributions)

Any other comments?

@Robert-Steiner Robert-Steiner changed the title only run workflow dispatch on default branch run workflow dispatch on default branch only Dec 18, 2023
@Robert-Steiner Robert-Steiner changed the title run workflow dispatch on default branch only Run workflow dispatch on default branch only Dec 18, 2023
@Robert-Steiner Robert-Steiner force-pushed the limit-branch-of-workflow-dispatch branch from db77aaa to 3000f7a Compare December 18, 2023 10:08
@Robert-Steiner Robert-Steiner changed the title Run workflow dispatch on default branch only Run only workflow dispatch on default branch Dec 18, 2023
@Robert-Steiner Robert-Steiner marked this pull request as ready for review December 18, 2023 11:55
@tanertopal tanertopal enabled auto-merge (squash) December 18, 2023 20:40
@tanertopal tanertopal merged commit fa7be1b into main Dec 18, 2023
27 checks passed
@tanertopal tanertopal deleted the limit-branch-of-workflow-dispatch branch December 18, 2023 22:14
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.

3 participants