Skip to content
This repository has been archived by the owner on Jul 1, 2024. It is now read-only.

Enforce remove/Recently Merged on close/merge #351

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 0 additions & 2 deletions src/_tests/fixtures/38979/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/43136/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/43144/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/43151/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/43160/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/43175/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/43235/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/43314/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/43695-duplicate-comment/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/43695-post-review/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/43695/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/43960-post-close/result.json
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
{
"labels": [],
"responseComments": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": false,
"projectColumn": "*REMOVE*"
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/43960/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/44105/result.json
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
{
"labels": [],
"responseComments": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": false,
"projectColumn": "*REMOVE*"
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/44256/result.json
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
{
"labels": [],
"responseComments": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": false,
"projectColumn": "*REMOVE*"
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/44267/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/44282/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/44288/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/44290/result.json
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
{
"labels": [],
"responseComments": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": false,
"projectColumn": "Needs Author Action"
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/44299-with-files/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/44299/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/44316/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/44343-pending-travis/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/44343-pre-travis/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/44343/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/44402/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/44411/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,5 @@
"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
}
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/44424-2-after-travis-second/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/44437/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/44439/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/44631/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/44857/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/44989-14days/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,5 @@
"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
}
5 changes: 2 additions & 3 deletions src/_tests/fixtures/44989-32days/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@
"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
"shouldUpdateLabels": true,
"state": "close"
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/44989-3days/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/44989-7days/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/45137/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,5 @@
"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
}
22 changes: 3 additions & 19 deletions src/_tests/fixtures/45627/mutations.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,10 +13,11 @@
}
},
{
"mutation": "mutation ($input: DeleteProjectCardInput!) {\n deleteProjectCard(input: $input) {\n __typename\n }\n}\n",
"mutation": "mutation ($input: MoveProjectCardInput!) {\n moveProjectCard(input: $input) {\n __typename\n }\n}\n",
"variables": {
"input": {
"cardId": "MDExOlByb2plY3RDYXJkNDA0NTk5NzM="
"cardId": "MDExOlByb2plY3RDYXJkNDA0NTk5NzM=",
"columnId": "MDEzOlByb2plY3RDb2x1bW43NTUyOTI1"
}
}
},
Expand Down Expand Up @@ -47,15 +48,6 @@
}
}
},
{
"mutation": "mutation ($input: AddCommentInput!) {\n addComment(input: $input) {\n __typename\n }\n}\n",
"variables": {
"input": {
"subjectId": "MDExOlB1bGxSZXF1ZXN0NDM3NTIxMTQ2",
"body": "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.\n<!--typescript_bot_Unmerged:done-->"
}
}
},
{
"mutation": "mutation ($input: MergePullRequestInput!) {\n mergePullRequest(input: $input) {\n __typename\n }\n}\n",
"variables": {
Expand All @@ -66,13 +58,5 @@
"pullRequestId": "MDExOlB1bGxSZXF1ZXN0NDM3NTIxMTQ2"
}
}
},
{
"mutation": "mutation ($input: ClosePullRequestInput!) {\n closePullRequest(input: $input) {\n __typename\n }\n}\n",
"variables": {
"input": {
"pullRequestId": "MDExOlB1bGxSZXF1ZXN0NDM3NTIxMTQ2"
}
}
}
]
11 changes: 3 additions & 8 deletions src/_tests/fixtures/45627/result.json
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
{
"projectColumn": "*REMOVE*",
"projectColumn": "Recently Merged",
"labels": [
"Critical package",
"Owner Approved",
Expand All @@ -20,13 +20,8 @@
{
"tag": "merge-offer",
"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.)"
},
{
"tag": "Unmerged:done",
"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": true,
"shouldUpdateLabels": true
"shouldUpdateLabels": true,
"state": "merge"
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/45836/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,5 @@
"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
}
2 changes: 0 additions & 2 deletions src/_tests/fixtures/45884/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,5 @@
"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
}
Loading