Skip to content

Skip merge requests in given states#118

Merged
spruce merged 2 commits intopiceaTech:masterfrom
mdbenito:fix/105-skip-closed-mr
Jan 10, 2022
Merged

Skip merge requests in given states#118
spruce merged 2 commits intopiceaTech:masterfrom
mdbenito:fix/105-skip-closed-mr

Conversation

@mdbenito
Copy link
Copy Markdown
Contributor

@mdbenito mdbenito commented Dec 5, 2021

This PR adds a new config variable skipMergeRequestStates which is an array of strings. Merge requests with a state listed in this array will not be migrated. Closes #105

@spruce
Copy link
Copy Markdown
Member

spruce commented Dec 6, 2021

Wouldn't this lead to the issue numbers diverging from github and gitlab?

@mdbenito
Copy link
Copy Markdown
Contributor Author

mdbenito commented Dec 6, 2021

AFAIK merge requests are numbered independently from issues in GitLab, but not in GitHub, where everything is an issue, so this shouldn't be a problem. It could mess things up if #121 is implemented as suggested there. In that case maybe one could create and delete issues instead.

@spruce spruce merged commit a92af49 into piceaTech:master Jan 10, 2022
@mdbenito mdbenito deleted the fix/105-skip-closed-mr branch January 11, 2022 08:43
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.

Avoid issues from closed merge requests

2 participants