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

Add config to provide git push options #3159

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

MatthewJohn
Copy link

I've created this change for myself (currently in testing) and created the PR to see if you believe it would be useful.

The rational behind this:

  • I use Gitlab primarily but replicate to Github.
  • Some projects use both Gitlab CI/CD and Github actions (mainly for testing/tagging etc. in Gitlab and then producing artifacts in Github).

When I merge into Gitlab, semantic release runs on main and performs the release. The release commit, by default, uses [skip ci], which is appropriate for Gitlab. Unfortunately, when the release commit is replicated to Github, it also respects this and skips Github actions.

This change means that I change the commit message, removing [skip ci] and use a git push option ci.skip (see https://docs.gitlab.com/ee/user/project/push_options.html), which will only skip the CI/CD pipeline in Gitlab.

I'm creating this PR to get feedback if it's a feature that you believe would be useful and, if so, I can tidy it up after testing, add tests etc. ready for proper review :)

Many thanks
Matt

@MatthewJohn MatthewJohn marked this pull request as draft January 25, 2024 05:59
@MatthewJohn MatthewJohn changed the title Draft: Add config to provide git push options Add config to provide git push options Jan 25, 2024
@Luckymhe
Copy link

How can I update branch. Master branch suddenly stop working

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.

None yet

2 participants