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

Put explanations next to suggestions #265

Draft
wants to merge 19 commits into
base: master
Choose a base branch
from
1 change: 1 addition & 0 deletions src/_tests/fixtures/38979/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@
"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.)"
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/43136/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@
"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?"
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/43144/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
"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.)"
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/43151/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@
"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...)"
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/43160/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@
"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!"
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/43175/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@
"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?"
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/43235/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@
"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."
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/43314/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@
"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...)"
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/43695-duplicate-comment/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@
"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?».)"
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/43695-post-review/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
"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!"
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/43695/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
"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!"
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/43960-post-close/result.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
{
"labels": [],
"responseComments": [],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": false,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/43960/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
"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!"
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/44105/result.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
{
"labels": [],
"responseComments": [],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": false,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/44256/result.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
{
"labels": [],
"responseComments": [],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": false,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/44267/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
"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 ..."
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/44282/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
"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...)"
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/44288/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@
"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."
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/44290/result.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
{
"labels": [],
"responseComments": [],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": false,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/44299-with-files/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@
"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...)"
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/44299/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@
"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...)"
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/44316/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
"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...)"
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/44343-pending-travis/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@
"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."
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/44343-pre-travis/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@
"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."
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/44343/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@
"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."
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/44402/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@
"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"
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/44411/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@
"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."
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@
"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."
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@
"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."
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/44437/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@
"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.)"
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/44439/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
"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."
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/44631/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
"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!"
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/44857/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@
"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.)"
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/44989-14days/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@
"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.)"
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/44989-32days/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@
"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."
}
],
"explanations": [],
"shouldClose": true,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/44989-3days/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
"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.)"
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/44989-7days/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
"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.)"
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/45137/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@
"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 ..."
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/45627/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@
"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."
}
],
"explanations": [],
"shouldClose": true,
"shouldMerge": true,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/45836/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
"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!"
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/45884/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@
"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.)"
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/45888/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@
"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."
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/45890/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
"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...)"
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/45946/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
"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...)"
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
1 change: 1 addition & 0 deletions src/_tests/fixtures/45982/result.json
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@
"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 )?/"
}
],
"explanations": [],
"shouldClose": false,
"shouldMerge": false,
"shouldUpdateLabels": true,
Expand Down
Loading