Merge pull request #118 from mdbenito/fix/105-skip-closed-mr

Skip merge requests in given states

Author: spruce

README.md

@@ -155,6 +155,10 @@ used to sidestep the problem where pull requests are rejected by GitHub if the f
 
 Filters all merge requests and issues by these labels. The applicable values can be found in the Gitlab API documentation for [issues](https://docs.gitlab.com/ee/api/issues.html#list-project-issues) and [merge requests](https://docs.gitlab.com/ee/api/merge_requests.html#list-merge-requests) respectively. Default is `null` which returns all issues/merge requests.
 
+### skipMergeRequestStates
+
+Merge requests in GitLab with any of the states listed in this array will not be transferred to GitHub (e.g. set to `['merged', 'closed']` to avoid creating issues for closed MRs whose branches have been deleted).
+
 ### skipMatchingComments
 
 This is an array (empty per default) that may contain string values. Any note/comment in any issue, that contains one or more of those string values, will be skipped (meaining not migrated). Note that this is case insensitive, therefore the string value `foo` would also lead to skipping notes containing a (sub)string `FOO`.

sample_settings.ts

@@ -42,6 +42,7 @@ export default {
   useReplacementIssuesForCreationFails: true,
   useIssuesForAllMergeRequests: false,
   filterByLabel: null,
+  skipMergeRequestStates: [],
   skipMatchingComments: [],
   mergeRequests: {
     logFile: './merge-requests.json',

src/index.ts

@@ -437,6 +437,10 @@ async function transferMergeRequests() {
       i => i.title.trim().includes(request.title.trim())
     );
     if (!githubRequest && !githubIssue) {
+      if (settings.skipMergeRequestStates.includes(request.state)) {
+          console.log(`Skipping MR ${request.iid} in "${request.state}" state: ${request.title}`)
+          continue;
+        }
       console.log(
         'Creating pull request: !' + request.iid + ' - ' + request.title
       );

src/settings.ts

@@ -22,6 +22,7 @@ export default interface Settings {
   useReplacementIssuesForCreationFails: boolean;
   useIssuesForAllMergeRequests: boolean;
   filterByLabel: string | null;
+  skipMergeRequestStates: string[];
   skipMatchingComments: string[];
   mergeRequests: {
     logFile: string;