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

lab mr merge does not respect delete source branch #830

Open
JohnVillalovos opened this issue May 2, 2022 · 3 comments
Open

lab mr merge does not respect delete source branch #830

JohnVillalovos opened this issue May 2, 2022 · 3 comments

Comments

@JohnVillalovos
Copy link
Contributor

JohnVillalovos commented May 2, 2022

I create an mr using lab mr create --remove-source-branch and it shows that the source branch will be removed.

Then I do lab mr merge and then it no longer shows it will remove the source branch.

Using lab version:

$ lab --version
lab version 0.24.0
@zampierilucas
Copy link
Collaborator

Hey @JohnVillalovos thanks for reporting that.

Unfortunately I count to reproduce your issue, I've also played with the project setting under Merge Request for the default of delete source branch on merge and saw no difference in it, also I've tested on lab version v0.24.0-3-g73bada2(a bit behind master), so It should have been fixed by another commit, can you test on your project with the latest master version or g73bada2?

If that solves the issue, let me know, as I will try to speed up a 0.25.0 release to get it fixed.

@JohnVillalovos
Copy link
Contributor Author

JohnVillalovos commented May 3, 2022

So I am doing this on an MR which is still running the CI. Not sure if that is how you tested it. So by doing lab mr merge it sets the MR to merge after the pipeline has passed.

@zampierilucas
Copy link
Collaborator

Ooh, yeah, I don't have the setting to wait for pipeline completion before merge turned on, I'll do a bit more testing and get back to you

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

No branches or pull requests

2 participants