diff --git a/src/_tests/fixtures/38979/result.json b/src/_tests/fixtures/38979/result.json index b5080e652..98b320bef 100644 --- a/src/_tests/fixtures/38979/result.json +++ b/src/_tests/fixtures/38979/result.json @@ -23,8 +23,6 @@ "status": "It has been more than two weeks and this PR still has no reviews.\n\nI'll bump it to the DT maintainer queue. Thank you for your patience, @ExE-Boss.\n\n(Ping @RReverser.)" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/43136/result.json b/src/_tests/fixtures/43136/result.json index 41273c6fc..68c4e008e 100644 --- a/src/_tests/fixtures/43136/result.json +++ b/src/_tests/fixtures/43136/result.json @@ -17,8 +17,6 @@ "status": "@RReverser Thank you for reviewing this PR! The author has pushed new commits since your last review. Could you take another look and submit a fresh review?" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/43144/result.json b/src/_tests/fixtures/43144/result.json index ba3d5303b..ddbf49593 100644 --- a/src/_tests/fixtures/43144/result.json +++ b/src/_tests/fixtures/43144/result.json @@ -14,8 +14,6 @@ "status": "@jeffreymeng Everything looks good here. Great job! I am ready to merge this PR (at f1f5c4b) on your behalf.\n\nIf you'd like that to happen, please post a comment saying:\n\n> Ready to merge\n\nand I'll merge this PR almost instantly. Thanks for helping out! :heart:\n\n(@pocesar: you can do this too.)" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/43151/result.json b/src/_tests/fixtures/43151/result.json index 15118ca39..3c879d94f 100644 --- a/src/_tests/fixtures/43151/result.json +++ b/src/_tests/fixtures/43151/result.json @@ -13,8 +13,6 @@ "status": "πŸ”” @adamzerella β€” you're the only owner, but it would still be good if you find someone to [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/43151/files) in the next few days, otherwise a maintainer will look at it. (And if you do find someone, maybe even recruit them to be a second owner to make future changes easier...)" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/43160/result.json b/src/_tests/fixtures/43160/result.json index e6a7a641e..7cd9fe106 100644 --- a/src/_tests/fixtures/43160/result.json +++ b/src/_tests/fixtures/43160/result.json @@ -19,8 +19,6 @@ "status": "@rikkertkoppes The CI build failed! Please [review the logs for more information](https://github.com/DefinitelyTyped/DefinitelyTyped/commit/22c73c88cc9c09efd4c2998ec360607dd4c36c2e/checks?check_suite_id=731664306).\r\n\r\nOnce you've pushed the fixes, the build will automatically re-run. Thanks!" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/43175/result.json b/src/_tests/fixtures/43175/result.json index 97905e9f0..86d383b13 100644 --- a/src/_tests/fixtures/43175/result.json +++ b/src/_tests/fixtures/43175/result.json @@ -22,8 +22,6 @@ "status": "@couven92 Thank you for reviewing this PR! The author has pushed new commits since your last review. Could you take another look and submit a fresh review?" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/43235/result.json b/src/_tests/fixtures/43235/result.json index d95c720fa..35f28a507 100644 --- a/src/_tests/fixtures/43235/result.json +++ b/src/_tests/fixtures/43235/result.json @@ -11,8 +11,6 @@ "status": "πŸ”” @Esri @bsvensson β€” please [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/43235/files) in the next few days. Be sure to explicitly select **`Approve`** or **`Request Changes`** in the GitHub UI so I know what's going on." } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/43314/result.json b/src/_tests/fixtures/43314/result.json index a7fd4d28b..378148abd 100644 --- a/src/_tests/fixtures/43314/result.json +++ b/src/_tests/fixtures/43314/result.json @@ -13,8 +13,6 @@ "status": "πŸ”” @metonym β€” you're the only owner, but it would still be good if you find someone to [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/43314/files) in the next few days, otherwise a maintainer will look at it. (And if you do find someone, maybe even recruit them to be a second owner to make future changes easier...)" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/43695-duplicate-comment/result.json b/src/_tests/fixtures/43695-duplicate-comment/result.json index 374edbf1a..fee9a3b5b 100644 --- a/src/_tests/fixtures/43695-duplicate-comment/result.json +++ b/src/_tests/fixtures/43695-duplicate-comment/result.json @@ -22,8 +22,6 @@ "status": "It has been more than two weeks and this PR still has no reviews.\n\nI'll bump it to the DT maintainer queue. Thank you for your patience, @alexandercerutti.\n\n(Ping Β«anyone?Β».)" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/43695-post-review/result.json b/src/_tests/fixtures/43695-post-review/result.json index 51109cabf..7908b155e 100644 --- a/src/_tests/fixtures/43695-post-review/result.json +++ b/src/_tests/fixtures/43695-post-review/result.json @@ -14,8 +14,6 @@ "status": "@alexandercerutti One or more reviewers has requested changes. Please address their comments. I'll be back once they sign off or you've pushed new commits. Thank you!" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/43695/result.json b/src/_tests/fixtures/43695/result.json index 8ee3dae2c..5fcb52eea 100644 --- a/src/_tests/fixtures/43695/result.json +++ b/src/_tests/fixtures/43695/result.json @@ -14,8 +14,6 @@ "status": "@alexandercerutti One or more reviewers has requested changes. Please address their comments. I'll be back once they sign off or you've pushed new commits. Thank you!" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/43960-post-close/result.json b/src/_tests/fixtures/43960-post-close/result.json index 3a62818e6..b221b127f 100644 --- a/src/_tests/fixtures/43960-post-close/result.json +++ b/src/_tests/fixtures/43960-post-close/result.json @@ -1,8 +1,6 @@ { "labels": [], "responseComments": [], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": false, "shouldUpdateProjectColumn": false, "shouldRemoveFromActiveColumns": true diff --git a/src/_tests/fixtures/43960/result.json b/src/_tests/fixtures/43960/result.json index 0bac2a8d8..1fa86da0f 100644 --- a/src/_tests/fixtures/43960/result.json +++ b/src/_tests/fixtures/43960/result.json @@ -14,8 +14,6 @@ "status": "@aaltepet One or more reviewers has requested changes. Please address their comments. I'll be back once they sign off or you've pushed new commits. Thank you!" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/44105/result.json b/src/_tests/fixtures/44105/result.json index 3a62818e6..b221b127f 100644 --- a/src/_tests/fixtures/44105/result.json +++ b/src/_tests/fixtures/44105/result.json @@ -1,8 +1,6 @@ { "labels": [], "responseComments": [], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": false, "shouldUpdateProjectColumn": false, "shouldRemoveFromActiveColumns": true diff --git a/src/_tests/fixtures/44256/result.json b/src/_tests/fixtures/44256/result.json index 3a62818e6..b221b127f 100644 --- a/src/_tests/fixtures/44256/result.json +++ b/src/_tests/fixtures/44256/result.json @@ -1,8 +1,6 @@ { "labels": [], "responseComments": [], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": false, "shouldUpdateProjectColumn": false, "shouldRemoveFromActiveColumns": true diff --git a/src/_tests/fixtures/44267/result.json b/src/_tests/fixtures/44267/result.json index d97c5d4aa..7b177c54a 100644 --- a/src/_tests/fixtures/44267/result.json +++ b/src/_tests/fixtures/44267/result.json @@ -14,8 +14,6 @@ "status": "@ErikMartensson Thank you for submitting this PR! I see this is your first time submitting to DefinitelyTyped πŸ‘‹ β€” I'm the local bot who will help you through the process of getting things through.\n\n***This is a live comment which I will keep updated.***\n\n## 1 package in this PR\n\n* `intercom-client` [on npm](https://www.npmjs.com/package/intercom-client), [on unpkg](https://unpkg.com/browse/intercom-client@latest/)\n - owner-approval: @malithrw\n\n## Code Reviews\n\nThis PR can be merged once it's reviewed by a DT maintainer.\n\n## Status\n\n * βœ… No merge conflicts\n * βœ… Continuous integration tests have passed\n * ❌ Only a DT maintainer can approve changes [without tests](https://github.com/DefinitelyTyped/DefinitelyTyped#user-content-test-editing-an-existing-package)\n\nOnce every item on this list is checked, I'll ask you for permission to merge and publish the changes.\n\n----------------------\n... diagnostics scrubbed ..." } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/44282/result.json b/src/_tests/fixtures/44282/result.json index 7df903081..5434c1010 100644 --- a/src/_tests/fixtures/44282/result.json +++ b/src/_tests/fixtures/44282/result.json @@ -14,8 +14,6 @@ "status": "πŸ”” @fishcharlie β€” you're the only owner, but it would still be good if you find someone to [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/44282/files) in the next few days, otherwise a maintainer will look at it. (And if you do find someone, maybe even recruit them to be a second owner to make future changes easier...)" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/44288/result.json b/src/_tests/fixtures/44288/result.json index ff4bc27fd..3a056500f 100644 --- a/src/_tests/fixtures/44288/result.json +++ b/src/_tests/fixtures/44288/result.json @@ -11,8 +11,6 @@ "status": "πŸ”” @gosticks β€” please [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/44288/files) in the next few days. Be sure to explicitly select **`Approve`** or **`Request Changes`** in the GitHub UI so I know what's going on." } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/44290/result.json b/src/_tests/fixtures/44290/result.json index c7c95575c..14b1de814 100644 --- a/src/_tests/fixtures/44290/result.json +++ b/src/_tests/fixtures/44290/result.json @@ -1,8 +1,6 @@ { "labels": [], "responseComments": [], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": false, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false, diff --git a/src/_tests/fixtures/44299-with-files/result.json b/src/_tests/fixtures/44299-with-files/result.json index c07fdb0bf..72c68091e 100644 --- a/src/_tests/fixtures/44299-with-files/result.json +++ b/src/_tests/fixtures/44299-with-files/result.json @@ -13,8 +13,6 @@ "status": "πŸ”” @geopic β€” you're the only owner, but it would still be good if you find someone to [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/44299/files) in the next few days, otherwise a maintainer will look at it. (And if you do find someone, maybe even recruit them to be a second owner to make future changes easier...)" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/44299/result.json b/src/_tests/fixtures/44299/result.json index c07fdb0bf..72c68091e 100644 --- a/src/_tests/fixtures/44299/result.json +++ b/src/_tests/fixtures/44299/result.json @@ -13,8 +13,6 @@ "status": "πŸ”” @geopic β€” you're the only owner, but it would still be good if you find someone to [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/44299/files) in the next few days, otherwise a maintainer will look at it. (And if you do find someone, maybe even recruit them to be a second owner to make future changes easier...)" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/44316/result.json b/src/_tests/fixtures/44316/result.json index 55e2ae95b..3ff058f5e 100644 --- a/src/_tests/fixtures/44316/result.json +++ b/src/_tests/fixtures/44316/result.json @@ -14,8 +14,6 @@ "status": "πŸ”” @mattleff β€” you're the only owner, but it would still be good if you find someone to [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/44316/files) in the next few days, otherwise a maintainer will look at it. (And if you do find someone, maybe even recruit them to be a second owner to make future changes easier...)" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/44343-pending-travis/result.json b/src/_tests/fixtures/44343-pending-travis/result.json index 4b4abbd85..1ee323661 100644 --- a/src/_tests/fixtures/44343-pending-travis/result.json +++ b/src/_tests/fixtures/44343-pending-travis/result.json @@ -11,8 +11,6 @@ "status": "πŸ”” @joeriks β€” please [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/44343/files) in the next few days. Be sure to explicitly select **`Approve`** or **`Request Changes`** in the GitHub UI so I know what's going on." } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": false, "shouldUpdateProjectColumn": false, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/44343-pre-travis/result.json b/src/_tests/fixtures/44343-pre-travis/result.json index 4b4abbd85..1ee323661 100644 --- a/src/_tests/fixtures/44343-pre-travis/result.json +++ b/src/_tests/fixtures/44343-pre-travis/result.json @@ -11,8 +11,6 @@ "status": "πŸ”” @joeriks β€” please [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/44343/files) in the next few days. Be sure to explicitly select **`Approve`** or **`Request Changes`** in the GitHub UI so I know what's going on." } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": false, "shouldUpdateProjectColumn": false, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/44343/result.json b/src/_tests/fixtures/44343/result.json index 909ba1ecb..2d54b6639 100644 --- a/src/_tests/fixtures/44343/result.json +++ b/src/_tests/fixtures/44343/result.json @@ -11,8 +11,6 @@ "status": "πŸ”” @joeriks β€” please [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/44343/files) in the next few days. Be sure to explicitly select **`Approve`** or **`Request Changes`** in the GitHub UI so I know what's going on." } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/44402/result.json b/src/_tests/fixtures/44402/result.json index cc4e244a3..7ae0865f0 100644 --- a/src/_tests/fixtures/44402/result.json +++ b/src/_tests/fixtures/44402/result.json @@ -16,8 +16,6 @@ "status": "@sandersn Everything looks good here. Great job! I am ready to merge this PR (at 5dfb994) on your behalf.\n\nIf you'd like that to happen, please post a comment saying:\n\n> Ready to merge\n\nand I'll merge this PR almost instantly. Thanks for helping out! :heart:\n" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/44411/result.json b/src/_tests/fixtures/44411/result.json index e5e343d02..cef212791 100644 --- a/src/_tests/fixtures/44411/result.json +++ b/src/_tests/fixtures/44411/result.json @@ -13,8 +13,6 @@ "status": "πŸ”” @dawnmist @erbridge β€” please [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/44411/files) in the next few days. Be sure to explicitly select **`Approve`** or **`Request Changes`** in the GitHub UI so I know what's going on." } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/44424-1-travis-instantly-finished/result.json b/src/_tests/fixtures/44424-1-travis-instantly-finished/result.json index bbf12029d..31e5386cf 100644 --- a/src/_tests/fixtures/44424-1-travis-instantly-finished/result.json +++ b/src/_tests/fixtures/44424-1-travis-instantly-finished/result.json @@ -13,8 +13,6 @@ "status": "πŸ”” @chrisbarker @rdepena @whyn07m3 @licui3936 β€” please [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/44424/files) in the next few days. Be sure to explicitly select **`Approve`** or **`Request Changes`** in the GitHub UI so I know what's going on." } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/44424-2-after-travis-second/result.json b/src/_tests/fixtures/44424-2-after-travis-second/result.json index 4c26fc90e..d72e2257d 100644 --- a/src/_tests/fixtures/44424-2-after-travis-second/result.json +++ b/src/_tests/fixtures/44424-2-after-travis-second/result.json @@ -13,8 +13,6 @@ "status": "πŸ”” @chrisbarker @rdepena @whyn07m3 @licui3936 β€” please [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/44424/files) in the next few days. Be sure to explicitly select **`Approve`** or **`Request Changes`** in the GitHub UI so I know what's going on." } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/44437/result.json b/src/_tests/fixtures/44437/result.json index b8894ffa7..1961f3900 100644 --- a/src/_tests/fixtures/44437/result.json +++ b/src/_tests/fixtures/44437/result.json @@ -16,8 +16,6 @@ "status": "@johnnyreilly Everything looks good here. Great job! I am ready to merge this PR (at eb92456) on your behalf.\n\nIf you'd like that to happen, please post a comment saying:\n\n> Ready to merge\n\nand I'll merge this PR almost instantly. Thanks for helping out! :heart:\n\n(@Berkays, @unindented, @kamontat, @theweirdone, @whoaa512: you can do this too.)" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/44439/result.json b/src/_tests/fixtures/44439/result.json index 7256583a5..33347d618 100644 --- a/src/_tests/fixtures/44439/result.json +++ b/src/_tests/fixtures/44439/result.json @@ -14,8 +14,6 @@ "status": "πŸ”” @lith-light-g β€” please [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/44439/files) in the next few days. Be sure to explicitly select **`Approve`** or **`Request Changes`** in the GitHub UI so I know what's going on." } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/44631/result.json b/src/_tests/fixtures/44631/result.json index 97cf27132..8661b637c 100644 --- a/src/_tests/fixtures/44631/result.json +++ b/src/_tests/fixtures/44631/result.json @@ -14,8 +14,6 @@ "status": "@mAAdhaTTah Unfortunately, this pull request currently has a merge conflict πŸ˜₯. Please update your PR branch to be up-to-date with respect to master. Have a nice day!" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/44857/result.json b/src/_tests/fixtures/44857/result.json index 3ecd03109..a7f51156b 100644 --- a/src/_tests/fixtures/44857/result.json +++ b/src/_tests/fixtures/44857/result.json @@ -23,8 +23,6 @@ "status": "It has been more than two weeks and this PR still has no reviews.\n\nI'll bump it to the DT maintainer queue. Thank you for your patience, @ExE-Boss.\n\n(Ping @Microsoft, @DefinitelyTyped, @jkomyno, @a-tarasyuk, @alvis, @r3nya, @btoueg, @brunoscheufler, @smac89, @touffy, @DeividasBakanas, @eyqs, @Flarna, @Hannes-Magnusson-CK, @KSXGitHub, @hoo29, @kjin, @ajafff, @islishude, @mwiktorczyk, @mohsen1, @n-e, @galkin, @parambirs, @eps1lon, @SimonSchick, @ThomasdenH, @WilcoBakker, @wwwy3y3, @samuela, @kuehlein, @j-oliveras, @bhongy, @chyzwar, @trivikr, @nguymin4, @yoursunny, @qwelias, @Ryan-Willpower, @peterblazejewicz, @addaleax, @JasonHK.)" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/44989-14days/result.json b/src/_tests/fixtures/44989-14days/result.json index c7ca3ae07..695da6b57 100644 --- a/src/_tests/fixtures/44989-14days/result.json +++ b/src/_tests/fixtures/44989-14days/result.json @@ -20,8 +20,6 @@ "status": "Re-ping @petr-motejlek / @TheHandsomeCoder, @donnut, @mdekrey, @sbking, @afharo, @teves-castro, @1M0reBug, @hojberg, @samsonkeung, @angeloocana, @raynerd, @moshensky, @ethanresnick, @deftomat, @blimusiek, @biern, @rayhaneh, @rgm, @drewwyatt, @jottenlips, @minitesh, @krantisinh, @pirix-gh, @brekk, @nemo108, @jituanlin, @Philippe-mills, @Saul-Mirone, @Nicholaiii:\n\nThis PR has been ready to merge for over a week, and I haven't seen any requests to merge it. I will close it on Jul 6th (in three weeks) if this doesn't happen.\n\n(If there's no reason to avoid merging it, please do so. Otherwise, if it shouldn't be merged or if it needs more time, please close it or turn it into a draft.)" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/44989-32days/result.json b/src/_tests/fixtures/44989-32days/result.json index 519b80538..f7d134273 100644 --- a/src/_tests/fixtures/44989-32days/result.json +++ b/src/_tests/fixtures/44989-32days/result.json @@ -20,9 +20,8 @@ "status": "After a month, no one has requested merging the PR 😞. I'm going to assume that the change is not wanted after all, and will therefore close it." } ], - "shouldClose": true, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, - "shouldRemoveFromActiveColumns": true + "shouldRemoveFromActiveColumns": true, + "state": "close" } diff --git a/src/_tests/fixtures/44989-3days/result.json b/src/_tests/fixtures/44989-3days/result.json index fc56c02e5..9b2daa469 100644 --- a/src/_tests/fixtures/44989-3days/result.json +++ b/src/_tests/fixtures/44989-3days/result.json @@ -15,8 +15,6 @@ "status": "@petr-motejlek Everything looks good here. Great job! I am ready to merge this PR (at 9ca6086) on your behalf.\n\nIf you'd like that to happen, please post a comment saying:\n\n> Ready to merge\n\nand I'll merge this PR almost instantly. Thanks for helping out! :heart:\n\n(@TheHandsomeCoder, @donnut, @mdekrey, @sbking, @afharo, @teves-castro, @1M0reBug, @hojberg, @samsonkeung, @angeloocana, @raynerd, @moshensky, @ethanresnick, @deftomat, @blimusiek, @biern, @rayhaneh, @rgm, @drewwyatt, @jottenlips, @minitesh, @krantisinh, @pirix-gh, @brekk, @nemo108, @jituanlin, @Philippe-mills, @Saul-Mirone, @Nicholaiii: you can do this too.)" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/44989-7days/result.json b/src/_tests/fixtures/44989-7days/result.json index 32abc71d5..1f39bb148 100644 --- a/src/_tests/fixtures/44989-7days/result.json +++ b/src/_tests/fixtures/44989-7days/result.json @@ -15,8 +15,6 @@ "status": "@petr-motejlek Everything looks good here. Great job! I am ready to merge this PR (at 9ca6086) on your behalf.\n\nIf you'd like that to happen, please post a comment saying:\n\n> Ready to merge\n\nand I'll merge this PR almost instantly. Thanks for helping out! :heart:\n\n(@TheHandsomeCoder, @donnut, @mdekrey, @sbking, @afharo, @teves-castro, @1M0reBug, @hojberg, @samsonkeung, @angeloocana, @raynerd, @moshensky, @ethanresnick, @deftomat, @blimusiek, @biern, @rayhaneh, @rgm, @drewwyatt, @jottenlips, @minitesh, @krantisinh, @pirix-gh, @brekk, @nemo108, @jituanlin, @Philippe-mills, @Saul-Mirone, @Nicholaiii: you can do this too.)" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/45137/result.json b/src/_tests/fixtures/45137/result.json index d2b05eaa5..6605989e2 100644 --- a/src/_tests/fixtures/45137/result.json +++ b/src/_tests/fixtures/45137/result.json @@ -17,8 +17,6 @@ "status": "@lirbank Thank you for submitting this PR!\n\n***This is a live comment which I will keep updated.***\n\n## 1 package in this PR\n\n* `mongodb` [on npm](https://www.npmjs.com/package/mongodb), [on unpkg](https://unpkg.com/browse/mongodb@latest/) (author is owner)\n - owner-approval: @LinusU\n - 1 removed owner: ✎@lirbank\n\n## Code Reviews\n\nThis PR can be merged once it's reviewed by a DT maintainer.\n\n## Status\n\n * βœ… No merge conflicts\n * βœ… Continuous integration tests have passed\n * ❌ Only a DT maintainer can approve changes [without tests](https://github.com/DefinitelyTyped/DefinitelyTyped#user-content-test-editing-an-existing-package)\n\nOnce every item on this list is checked, I'll ask you for permission to merge and publish the changes.\n\n----------------------\n... diagnostics scrubbed ..." } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/45627/result.json b/src/_tests/fixtures/45627/result.json index 8d7c0b16f..f21715598 100644 --- a/src/_tests/fixtures/45627/result.json +++ b/src/_tests/fixtures/45627/result.json @@ -22,9 +22,8 @@ "status": "@spamshaker Everything looks good here. Great job! I am ready to merge this PR (at 15facc1) on your behalf.\n\nIf you'd like that to happen, please post a comment saying:\n\n> Ready to merge\n\nand I'll merge this PR almost instantly. Thanks for helping out! :heart:\n\n(@tkqubo, @bumbleblym, @bcherny, @tommytroylin, @mohsen1, @jcreamer898, @alan-agius4, @dennispg, @christophehurpeau, @ZSkycat, @johnnyreilly, @rwaskiewicz, @kuehlein, @grgur, @rubenspgcavalcante, @andersk, @ofhouse, @danielthank, @sasurau4, @dionshihk, @peterblazejewicz: you can do this too.)" } ], - "shouldClose": false, - "shouldMerge": true, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, - "shouldRemoveFromActiveColumns": false + "shouldRemoveFromActiveColumns": false, + "state": "merge" } diff --git a/src/_tests/fixtures/45836/result.json b/src/_tests/fixtures/45836/result.json index f1b7092cd..b8a0ff87d 100644 --- a/src/_tests/fixtures/45836/result.json +++ b/src/_tests/fixtures/45836/result.json @@ -15,8 +15,6 @@ "status": ":passport_control: Hi @mmorearty,\n\nI can't [accept a merge request](https://github.com/DefinitelyTyped/DefinitelyTyped#user-content-make-a-pull-request) until the PR has a green CI and was appropriately reviewed. I will let you know once that happens.\n\nThanks, and happy typing!" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/45884/result.json b/src/_tests/fixtures/45884/result.json index a16bdbf25..29af5b7ea 100644 --- a/src/_tests/fixtures/45884/result.json +++ b/src/_tests/fixtures/45884/result.json @@ -19,8 +19,6 @@ "status": "@sgratzl Everything looks good here. Great job! I am ready to merge this PR (at 1dcf44a) on your behalf.\n\nIf you'd like that to happen, please post a comment saying:\n\n> Ready to merge\n\nand I'll merge this PR almost instantly. Thanks for helping out! :heart:\n\n(@phreed, @wy193777, @ypconstante, @janniclas, @cerberuser, @gsbelarus, @peterjferrarotto, @spaxe, @appleparan, @Veckodag: you can do this too.)" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/45888/result.json b/src/_tests/fixtures/45888/result.json index 6d08fb4a6..b42f035de 100644 --- a/src/_tests/fixtures/45888/result.json +++ b/src/_tests/fixtures/45888/result.json @@ -18,8 +18,6 @@ "status": "πŸ”” @sampsonjoliver @andipaetzold @JiriBalcar @CodeLenny β€” please [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/45888/files) in the next few days. Be sure to explicitly select **`Approve`** or **`Request Changes`** in the GitHub UI so I know what's going on." } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/45890/result.json b/src/_tests/fixtures/45890/result.json index f37164f96..3aaf1015b 100644 --- a/src/_tests/fixtures/45890/result.json +++ b/src/_tests/fixtures/45890/result.json @@ -14,8 +14,6 @@ "status": "πŸ”” @dimkirt β€” you're the only owner, but it would still be good if you find someone to [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/45890/files) in the next few days, otherwise a maintainer will look at it. (And if you do find someone, maybe even recruit them to be a second owner to make future changes easier...)" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/45946/result.json b/src/_tests/fixtures/45946/result.json index fcd635822..c98b43c22 100644 --- a/src/_tests/fixtures/45946/result.json +++ b/src/_tests/fixtures/45946/result.json @@ -14,8 +14,6 @@ "status": "πŸ”” @rubensworks β€” you're the only owner, but it would still be good if you find someone to [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/45946/files) in the next few days, otherwise a maintainer will look at it. (And if you do find someone, maybe even recruit them to be a second owner to make future changes easier...)" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/45982/result.json b/src/_tests/fixtures/45982/result.json index 42b94acd7..9a021f6e1 100644 --- a/src/_tests/fixtures/45982/result.json +++ b/src/_tests/fixtures/45982/result.json @@ -9,8 +9,6 @@ "status": "@dasa β€” There was an error that prevented me from properly processing this PR:\n\n error parsing owners: At 1:1 : Expected /\\/\\/ Type definitions for (non-npm package )?/" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/45999/result.json b/src/_tests/fixtures/45999/result.json index b6d714f80..17b485d4c 100644 --- a/src/_tests/fixtures/45999/result.json +++ b/src/_tests/fixtures/45999/result.json @@ -15,8 +15,6 @@ "status": "@alexpyzhianov Everything looks good here. Great job! I am ready to merge this PR (at 381a2a9) on your behalf.\n\nIf you'd like that to happen, please post a comment saying:\n\n> Ready to merge\n\nand I'll merge this PR almost instantly. Thanks for helping out! :heart:\n\n(@johnnyreilly, @bbenezech, @pzavolinsky, @digiguru, @ericanderson, @DovydasNavickas, @theruther4d, @guilhermehubner, @ferdaber, @jrakotoharisoa, @pascaloliv, @hotell, @franklixuefei, @Jessidhia, @saranshkataria, @lukyth, @eps1lon, @zieka, @dancerphil, @dimitropoulos, @disjukr, @vhfmag, @hellatan: you can do this too.)" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/46008/result.json b/src/_tests/fixtures/46008/result.json index a7a7c3271..7684eae33 100644 --- a/src/_tests/fixtures/46008/result.json +++ b/src/_tests/fixtures/46008/result.json @@ -19,8 +19,6 @@ "status": "@risingBirdSong Everything looks good here. Great job! I am ready to merge this PR (at 3e19cb9) on your behalf.\n\nIf you'd like that to happen, please post a comment saying:\n\n> Ready to merge\n\nand I'll merge this PR almost instantly. Thanks for helping out! :heart:\n\n(@p5-types, @Zalastax: you can do this too.)" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/46019/result.json b/src/_tests/fixtures/46019/result.json index b84054eed..ab8d5a739 100644 --- a/src/_tests/fixtures/46019/result.json +++ b/src/_tests/fixtures/46019/result.json @@ -15,8 +15,6 @@ "status": "@peterblazejewicz Everything looks good here. Great job! I am ready to merge this PR (at ceca9f7) on your behalf.\n\nIf you'd like that to happen, please post a comment saying:\n\n> Ready to merge\n\nand I'll merge this PR almost instantly. Thanks for helping out! :heart:\n" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/46120/result.json b/src/_tests/fixtures/46120/result.json index 77b39e27b..3acc8cbc2 100644 --- a/src/_tests/fixtures/46120/result.json +++ b/src/_tests/fixtures/46120/result.json @@ -20,8 +20,6 @@ "status": ":passport_control: Hi @reubenrybnik,\n\nI can't [accept a merge request](https://github.com/DefinitelyTyped/DefinitelyTyped#user-content-make-a-pull-request) until the PR has a green CI and was appropriately reviewed. I will let you know once that happens.\n\nThanks, and happy typing!" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/46191/result.json b/src/_tests/fixtures/46191/result.json index 7f34630ce..f23a9dafb 100644 --- a/src/_tests/fixtures/46191/result.json +++ b/src/_tests/fixtures/46191/result.json @@ -23,8 +23,6 @@ "status": "@jordanoverbye The CI build failed! Please [review the logs for more information](https://github.com/DefinitelyTyped/DefinitelyTyped/commit/3cc81dbde57a1b0eda6f69f539fa49b8d420adff/checks?check_suite_id=938074936).\r\n\r\nOnce you've pushed the fixes, the build will automatically re-run. Thanks!" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/46196/result.json b/src/_tests/fixtures/46196/result.json index ffe777f34..fef9a1f1b 100644 --- a/src/_tests/fixtures/46196/result.json +++ b/src/_tests/fixtures/46196/result.json @@ -11,8 +11,6 @@ "status": "πŸ”” @HeeL β€” please [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/46196/files) in the next few days. Be sure to explicitly select **`Approve`** or **`Request Changes`** in the GitHub UI so I know what's going on." } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/46279/result.json b/src/_tests/fixtures/46279/result.json index a7d8c8606..0d686a5aa 100644 --- a/src/_tests/fixtures/46279/result.json +++ b/src/_tests/fixtures/46279/result.json @@ -19,8 +19,6 @@ "status": "πŸ”” @pzingg β€” you're the only owner, but it would still be good if you find someone to [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/46279/files) in the next few days, otherwise a maintainer will look at it. (And if you do find someone, maybe even recruit them to be a second owner to make future changes easier...)" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/46804/result.json b/src/_tests/fixtures/46804/result.json index ab7e7840b..7ba914a25 100644 --- a/src/_tests/fixtures/46804/result.json +++ b/src/_tests/fixtures/46804/result.json @@ -17,8 +17,6 @@ "status": "πŸ”” @hrngoode @adina-todoran @BreadAndRoses95 β€” please [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/46804/files) in the next few days. Be sure to explicitly select **`Approve`** or **`Request Changes`** in the GitHub UI so I know what's going on." } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/46879/result.json b/src/_tests/fixtures/46879/result.json index 4b87b1d5a..4ad7a8231 100644 --- a/src/_tests/fixtures/46879/result.json +++ b/src/_tests/fixtures/46879/result.json @@ -13,8 +13,6 @@ "status": "πŸ”” @sudoplz @GiedriusGrabauskas β€” please [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/46879/files) in the next few days. Be sure to explicitly select **`Approve`** or **`Request Changes`** in the GitHub UI so I know what's going on." } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/47017-blessed-and-one-owner/result.json b/src/_tests/fixtures/47017-blessed-and-one-owner/result.json index 14288e838..39a940424 100644 --- a/src/_tests/fixtures/47017-blessed-and-one-owner/result.json +++ b/src/_tests/fixtures/47017-blessed-and-one-owner/result.json @@ -10,8 +10,6 @@ "status": "@mastermatt Thank you for submitting this PR!\n\n***This is a live comment which I will keep updated.***\n\n## 2 packages in this PR\n\n* `express-serve-static-core` [on npm](https://www.npmjs.com/package/express-serve-static-core), [on unpkg](https://unpkg.com/browse/express-serve-static-core@latest/)\n - owner-approval: @dwrss\n* `express` [on npm](https://www.npmjs.com/package/express), [on unpkg](https://unpkg.com/browse/express@latest/)\n\n## Code Reviews\n\nThis PR can be merged once it's reviewed.\n\n## Status\n\n * βœ… No merge conflicts\n * βœ… Continuous integration tests have passed\n * ❌ All owners or a DT maintainer needs to approve changes which affect more than one package\n - βœ… express-serve-static-core\n - ❌ express\n\nOnce every item on this list is checked, I'll ask you for permission to merge and publish the changes.\n\n----------------------\n... diagnostics scrubbed ..." } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/47017-blessed-and-two-owner/result.json b/src/_tests/fixtures/47017-blessed-and-two-owner/result.json index f68fc505e..6aae66049 100644 --- a/src/_tests/fixtures/47017-blessed-and-two-owner/result.json +++ b/src/_tests/fixtures/47017-blessed-and-two-owner/result.json @@ -16,8 +16,6 @@ "status": "@mastermatt Everything looks good here. Great job! I am ready to merge this PR (at dbe687d) on your behalf.\n\nIf you'd like that to happen, please post a comment saying:\n\n> Ready to merge\n\nand I'll merge this PR almost instantly. Thanks for helping out! :heart:\n" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/47017-blessed/result.json b/src/_tests/fixtures/47017-blessed/result.json index 60e0393c7..9c3fdaa54 100644 --- a/src/_tests/fixtures/47017-blessed/result.json +++ b/src/_tests/fixtures/47017-blessed/result.json @@ -14,8 +14,6 @@ "status": "πŸ”” @borisyankov @19majkel94 @kacepe @micksatana @samijaber @aereal @JoseLion @dwrss @andoshin11 @CMUH @puneetar @dfrankland β€” please [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/47017/files) in the next few days. Be sure to explicitly select **`Approve`** or **`Request Changes`** in the GitHub UI so I know what's going on." } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/47017/result.json b/src/_tests/fixtures/47017/result.json index 69cc7992c..0bad1dd3f 100644 --- a/src/_tests/fixtures/47017/result.json +++ b/src/_tests/fixtures/47017/result.json @@ -14,8 +14,6 @@ "status": "πŸ”” @borisyankov @19majkel94 @kacepe @micksatana @samijaber @aereal @JoseLion @dwrss @andoshin11 @CMUH @puneetar @dfrankland β€” please [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/47017/files) in the next few days. Be sure to explicitly select **`Approve`** or **`Request Changes`** in the GitHub UI so I know what's going on." } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/48216/result.json b/src/_tests/fixtures/48216/result.json index 9ba676916..b9eaad9fe 100644 --- a/src/_tests/fixtures/48216/result.json +++ b/src/_tests/fixtures/48216/result.json @@ -11,8 +11,6 @@ "status": "πŸ”” @innovation-team @rlbaxter β€” please [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/48216/files) in the next few days. Be sure to explicitly select **`Approve`** or **`Request Changes`** in the GitHub UI so I know what's going on." } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/48236/result.json b/src/_tests/fixtures/48236/result.json index 21950903b..f0e1e20ba 100644 --- a/src/_tests/fixtures/48236/result.json +++ b/src/_tests/fixtures/48236/result.json @@ -18,9 +18,8 @@ "status": "@jablko Everything looks good here. Great job! I am ready to merge this PR (at b4d71f6) on your behalf.\n\nIf you'd like that to happen, please post a comment saying:\n\n> Ready to merge\n\nand I'll merge this PR almost instantly. Thanks for helping out! :heart:\n\n(@climba03003: you can do this too.)" } ], - "shouldClose": false, - "shouldMerge": true, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, - "shouldRemoveFromActiveColumns": false + "shouldRemoveFromActiveColumns": false, + "state": "merge" } diff --git a/src/_tests/fixtures/48652-merge-offer/result.json b/src/_tests/fixtures/48652-merge-offer/result.json index aa5a7f6ce..fa748ffb7 100644 --- a/src/_tests/fixtures/48652-merge-offer/result.json +++ b/src/_tests/fixtures/48652-merge-offer/result.json @@ -20,8 +20,6 @@ "status": "@RyanCavanaugh, @thorn0, @falsyvalues Thank you for reviewing this PR! The author has pushed new commits since your last review. Could you take another look and submit a fresh review?" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/48652-prereq/result.json b/src/_tests/fixtures/48652-prereq/result.json index 8d3caa424..512243b3f 100644 --- a/src/_tests/fixtures/48652-prereq/result.json +++ b/src/_tests/fixtures/48652-prereq/result.json @@ -24,8 +24,6 @@ "status": ":passport_control: Hi @mgol,\n\nI can't [accept a merge request](https://github.com/DefinitelyTyped/DefinitelyTyped#user-content-make-a-pull-request) until the PR has a green CI and was appropriately reviewed. I will let you know once that happens.\n\nThanks, and happy typing!" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/48652-retract-merge-offer-and-prerequest/result.json b/src/_tests/fixtures/48652-retract-merge-offer-and-prerequest/result.json index c7a627d0a..1a7d8c0aa 100644 --- a/src/_tests/fixtures/48652-retract-merge-offer-and-prerequest/result.json +++ b/src/_tests/fixtures/48652-retract-merge-offer-and-prerequest/result.json @@ -18,8 +18,6 @@ "status": ":passport_control: Hi @mgol,\n\nI can't [accept a merge request](https://github.com/DefinitelyTyped/DefinitelyTyped#user-content-make-a-pull-request) until the PR has a green CI and was appropriately reviewed. I will let you know once that happens.\n\nThanks, and happy typing!" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/48652-retract-merge-offer/result.json b/src/_tests/fixtures/48652-retract-merge-offer/result.json index 6e9648234..28588e74d 100644 --- a/src/_tests/fixtures/48652-retract-merge-offer/result.json +++ b/src/_tests/fixtures/48652-retract-merge-offer/result.json @@ -14,8 +14,6 @@ "status": "@mgol One or more reviewers has requested changes. Please address their comments. I'll be back once they sign off or you've pushed new commits. Thank you!" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/48708/result.json b/src/_tests/fixtures/48708/result.json index 75d99cc91..ebbce4499 100644 --- a/src/_tests/fixtures/48708/result.json +++ b/src/_tests/fixtures/48708/result.json @@ -20,8 +20,6 @@ "status": "@martin-badin I haven't seen any activity on this PR in more than three weeks, and it still has problems that prevent it from being merged. The PR will be closed on Dec 12th (in a week) if the issues aren't addressed." } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/48945/result.json b/src/_tests/fixtures/48945/result.json index 639edf992..366c81a12 100644 --- a/src/_tests/fixtures/48945/result.json +++ b/src/_tests/fixtures/48945/result.json @@ -14,8 +14,6 @@ "status": "@google-api-typings-generator One or more reviewers has requested changes. Please address their comments. I'll be back once they sign off or you've pushed new commits. Thank you!" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/49417/result.json b/src/_tests/fixtures/49417/result.json index 7060b732e..517bba243 100644 --- a/src/_tests/fixtures/49417/result.json +++ b/src/_tests/fixtures/49417/result.json @@ -19,8 +19,6 @@ "status": "@tuuling Thank you for reviewing this PR! The author has pushed new commits since your last review. Could you take another look and submit a fresh review?" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/49548/result.json b/src/_tests/fixtures/49548/result.json index 8a9272a2a..55a506abc 100644 --- a/src/_tests/fixtures/49548/result.json +++ b/src/_tests/fixtures/49548/result.json @@ -19,8 +19,6 @@ "status": "Re-ping Β«anyone?Β»:\n\nThis PR has been out for over a week, yet I haven't seen any reviews.\n\nCould someone please give it some attention? Thanks!" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/49575/result.json b/src/_tests/fixtures/49575/result.json index b9a794fe4..ce05370ac 100644 --- a/src/_tests/fixtures/49575/result.json +++ b/src/_tests/fixtures/49575/result.json @@ -18,8 +18,6 @@ "status": "@elibarzilay, @sandersn Thank you for reviewing this PR! The author has pushed new commits since your last review. Could you take another look and submit a fresh review?" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/49841/result.json b/src/_tests/fixtures/49841/result.json index c426d5844..aa78f648a 100644 --- a/src/_tests/fixtures/49841/result.json +++ b/src/_tests/fixtures/49841/result.json @@ -19,8 +19,6 @@ "status": "@shekari-ah The CI build failed! Please [review the logs for more information](https://github.com/DefinitelyTyped/DefinitelyTyped/commit/4849e8876b0ab7adc889ccc35b55fde6f0274837/checks?check_suite_id=1574877739).\r\n\r\nOnce you've pushed the fixes, the build will automatically re-run. Thanks!" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/50429/result.json b/src/_tests/fixtures/50429/result.json index c2c19a9e0..4ac75b53b 100644 --- a/src/_tests/fixtures/50429/result.json +++ b/src/_tests/fixtures/50429/result.json @@ -19,8 +19,6 @@ "status": "@SaltyAom The CI build failed! Please [review the logs for more information](https://github.com/DefinitelyTyped/DefinitelyTyped/commit/8454d0853c273f9a20b08bd68cfd5c6b4f70c018/checks?check_suite_id=1778052567).\r\n\r\nOnce you've pushed the fixes, the build will automatically re-run. Thanks!" } ], - "shouldClose": false, - "shouldMerge": false, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, "shouldRemoveFromActiveColumns": false diff --git a/src/_tests/fixtures/50443/result.json b/src/_tests/fixtures/50443/result.json index 302c1585b..726434dd5 100644 --- a/src/_tests/fixtures/50443/result.json +++ b/src/_tests/fixtures/50443/result.json @@ -15,9 +15,8 @@ "status": "@shockdevv Everything looks good here. Great job! I am ready to merge this PR (at 130c088) on your behalf.\n\nIf you'd like that to happen, please post a comment saying:\n\n> Ready to merge\n\nand I'll merge this PR almost instantly. Thanks for helping out! :heart:\n" } ], - "shouldClose": false, - "shouldMerge": true, "shouldUpdateLabels": true, "shouldUpdateProjectColumn": true, - "shouldRemoveFromActiveColumns": false + "shouldRemoveFromActiveColumns": false, + "state": "merge" }