DefinitelyTyped / dt-mergebot

The bot which handles auto-merging your PRs
https://devblogs.microsoft.com/typescript/changes-to-how-we-manage-definitelytyped/
MIT License
112 stars 44 forks source link

Ignore nonexistent PR numbers #379

Closed elibarzilay closed 3 years ago

elibarzilay commented 3 years ago

Make prFromCheckSuiteEvent return a new ignore-this value when we couldn't find a PR and there are check_suite.pull_requests. From looking at the logs, it seems to me that all of them are "reverse" PRs (used by forks to pull updates from DT, mostly by a "pull bot").

Keep throwing an error if no PR was found and there are no such reverse PRs.