diff --git a/lib/pr_checker.js b/lib/pr_checker.js index b37314e4..754a89c2 100644 --- a/lib/pr_checker.js +++ b/lib/pr_checker.js @@ -29,7 +29,6 @@ const GITHUB_SUCCESS_CONCLUSIONS = ['SUCCESS', 'NEUTRAL', 'SKIPPED']; const FAST_TRACK_RE = /^Fast-track has been requested by @(.+?)\. Please 👍 to approve\.$/; const FAST_TRACK_MIN_APPROVALS = 2; const GIT_CONFIG_GUIDE_URL = 'https://github.com/nodejs/node/blob/99b1ada/doc/guides/contributing/pull-requests.md#step-1-fork'; -const IGNORED_CHECK_SLUGS = ['dependabot', 'codecov']; // eslint-disable-next-line no-extend-native Array.prototype.findLastIndex ??= function findLastIndex(fn) { @@ -374,9 +373,10 @@ export default class PRChecker { // GitHub new Check API for (const { status, conclusion, app } of checkSuites.nodes) { - if (app && IGNORED_CHECK_SLUGS.includes(app.slug)) { - // Ignore Dependabot and Codecov check suites. - // They are expected to show up sometimes and never complete. + if (app.slug !== 'github-actions') { + // Ignore all non-github check suites, such as Dependabot and Codecov. + // They are expected to show up on PRs whose head branch is not on a + // fork and never complete. continue; } diff --git a/test/fixtures/github-ci/both-apis-failure.json b/test/fixtures/github-ci/both-apis-failure.json index 3866f56d..aa552cce 100644 --- a/test/fixtures/github-ci/both-apis-failure.json +++ b/test/fixtures/github-ci/both-apis-failure.json @@ -13,6 +13,7 @@ "checkSuites": { "nodes": [ { + "app": { "slug": "github-actions" }, "status": "COMPLETED", "conclusion": "FAILURE" } diff --git a/test/fixtures/github-ci/both-apis-success.json b/test/fixtures/github-ci/both-apis-success.json index 86fb337c..c58e7abc 100644 --- a/test/fixtures/github-ci/both-apis-success.json +++ b/test/fixtures/github-ci/both-apis-success.json @@ -13,6 +13,7 @@ "checkSuites": { "nodes": [ { + "app": { "slug": "github-actions" }, "status": "COMPLETED", "conclusion": "SUCCESS" } diff --git a/test/fixtures/github-ci/check-suite-failure.json b/test/fixtures/github-ci/check-suite-failure.json index 9f34c110..b299e1e7 100644 --- a/test/fixtures/github-ci/check-suite-failure.json +++ b/test/fixtures/github-ci/check-suite-failure.json @@ -10,6 +10,7 @@ "checkSuites": { "nodes": [ { + "app": { "slug": "github-actions" }, "status": "COMPLETED", "conclusion": "FAILURE" } diff --git a/test/fixtures/github-ci/check-suite-pending.json b/test/fixtures/github-ci/check-suite-pending.json index de284a9a..c09d11bf 100644 --- a/test/fixtures/github-ci/check-suite-pending.json +++ b/test/fixtures/github-ci/check-suite-pending.json @@ -10,6 +10,7 @@ "checkSuites": { "nodes": [ { + "app": { "slug": "github-actions" }, "status": "IN_PROGRESS" } ] diff --git a/test/fixtures/github-ci/check-suite-skipped.json b/test/fixtures/github-ci/check-suite-skipped.json index 42012526..6a2cb755 100644 --- a/test/fixtures/github-ci/check-suite-skipped.json +++ b/test/fixtures/github-ci/check-suite-skipped.json @@ -10,6 +10,7 @@ "checkSuites": { "nodes": [ { + "app": { "slug": "github-actions" }, "status": "COMPLETED", "conclusion": "SKIPPED" } diff --git a/test/fixtures/github-ci/check-suite-success.json b/test/fixtures/github-ci/check-suite-success.json index ed7f3d6b..1dc5f395 100644 --- a/test/fixtures/github-ci/check-suite-success.json +++ b/test/fixtures/github-ci/check-suite-success.json @@ -10,6 +10,7 @@ "checkSuites": { "nodes": [ { + "app": { "slug": "github-actions" }, "status": "COMPLETED", "conclusion": "SUCCESS" } diff --git a/test/fixtures/github-ci/status-failure-check-suite-succeed.json b/test/fixtures/github-ci/status-failure-check-suite-succeed.json index 98b03e69..28faa58c 100644 --- a/test/fixtures/github-ci/status-failure-check-suite-succeed.json +++ b/test/fixtures/github-ci/status-failure-check-suite-succeed.json @@ -13,6 +13,7 @@ "checkSuites": { "nodes": [ { + "app": { "slug": "github-actions" }, "status": "COMPLETED", "conclusion": "SUCCESS" } diff --git a/test/fixtures/github-ci/status-succeed-check-suite-failure.json b/test/fixtures/github-ci/status-succeed-check-suite-failure.json index cef60ce6..6a336cd8 100644 --- a/test/fixtures/github-ci/status-succeed-check-suite-failure.json +++ b/test/fixtures/github-ci/status-succeed-check-suite-failure.json @@ -13,6 +13,7 @@ "checkSuites": { "nodes": [ { + "app": { "slug": "github-actions" }, "status": "COMPLETED", "conclusion": "FAILURE" } diff --git a/test/fixtures/github-ci/success-dependabot-queued.json b/test/fixtures/github-ci/success-dependabot-queued.json index a798e9f9..557452f5 100644 --- a/test/fixtures/github-ci/success-dependabot-queued.json +++ b/test/fixtures/github-ci/success-dependabot-queued.json @@ -17,6 +17,7 @@ "conclusion": null }, { + "app": { "slug": "github-actions" }, "status": "COMPLETED", "conclusion": "SUCCESS" }