"doNotNotify" -> "do_not_notify"

metadata.json already has the field "yanked_versions" using lower snake case, so I updated this field to follow suit.
bazelbuild · Sep 17, 2024 · 0a6d4b2 · 0a6d4b2
1 parent d0a648c
commit 0a6d4b2
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/actions/bcr-pr-reviewer/index.js b/actions/bcr-pr-reviewer/index.js
@@ -45,8 +45,8 @@ async function generateMaintainersMap(octokit, owner, repo, modifiedModules, toN
       const metadata = JSON.parse(Buffer.from(metadataContent.content, 'base64').toString('utf-8'));
       let hasGithubMaintainer = false;
       metadata.maintainers.forEach(maintainer => {
-        // Only add maintainers with a github handle set. When `toNotifyOnly`, also exclude those who have set "doNotNotify"
-        if (maintainer.github && !(toNotifyOnly && maintainer.doNotNotify)) {
+        // Only add maintainers with a github handle set. When `toNotifyOnly`, also exclude those who have set "do_not_notify"
+        if (maintainer.github && !(toNotifyOnly && maintainer["do_not_notify"])) {
           hasGithubMaintainer = true;
           if (!maintainersMap.has(maintainer.github)) {
             maintainersMap.set(maintainer.github, new Set());