From 0d7db99d7f02be330f51e85af7086c096cf61ff5 Mon Sep 17 00:00:00 2001 From: William Chargin Date: Tue, 10 Sep 2019 08:55:16 -0700 Subject: [PATCH] Blacklist `@allcontributors` bot (#1363) Summary: This adds `MDM6Qm90NDY0NDczMjE=` (`@allcontributors`) to the blacklist to enable loading the `aragon/aragon` repository. See #1362 and #996 for context. Test Plan: Running `node ./bin/sourcecred.js load aragon/aragon` on a clean cache now completes successfully. wchargin-branch: blacklist-allcontributors --- src/plugins/github/blacklistedObjectIds.js | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/plugins/github/blacklistedObjectIds.js b/src/plugins/github/blacklistedObjectIds.js index 41f1f72..3678463 100644 --- a/src/plugins/github/blacklistedObjectIds.js +++ b/src/plugins/github/blacklistedObjectIds.js @@ -11,10 +11,11 @@ export const BLACKLISTED_IDS: $ReadOnlyArray = deepFreeze([ "MDEyOk9yZ2FuaXphdGlvbjI5MTkzOTQ=", "MDEyOk9yZ2FuaXphdGlvbjEyNDE3MDI0", "MDEyOk9yZ2FuaXphdGlvbjQzMDkzODIw", - // In this case, the bot used to be a user (@greenkeeper) - "MDM6Qm90MjMwNDAwNzY=", - // @dependabot also gives incosistent results (user vs bot) - "MDM6Qm90NDk2OTkzMzM=", + // These are `Bot` nodes that are sometimes referenced in a `User` + // context: in particular, as the author of a commit. + "MDM6Qm90MjMwNDAwNzY=", // greenkeeper + "MDM6Qm90NDk2OTkzMzM=", // dependabot + "MDM6Qm90NDY0NDczMjE=", // allcontributors // These are the offending reactions. "MDg6UmVhY3Rpb24yMTY3ODkyNQ==", "MDg6UmVhY3Rpb240NDMwMzQ1",