From 98a34e0c4ee9c244d9653d90f6462db93909d7a0 Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Wed, 22 Dec 2021 22:43:00 -0800 Subject: [PATCH] tools: use {N} for spaces in regex Spaces are hard to count. Use {N} notation to indicate how many spaces in regular expressions in find-inactive-collaborators. --- tools/find-inactive-collaborators.mjs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tools/find-inactive-collaborators.mjs b/tools/find-inactive-collaborators.mjs index 4d7802a0ed30f5..e01b29ac5f9f23 100755 --- a/tools/find-inactive-collaborators.mjs +++ b/tools/find-inactive-collaborators.mjs @@ -49,7 +49,7 @@ const authors = await runGitCommand( // Get all approving reviewers of landed commits during the time period. const approvingReviewers = await runGitCommand( `git log --since="${SINCE}" | egrep "^ Reviewed-By: "`, - (line) => /^ Reviewed-By: ([^<]+)/.exec(line)[1].trim() + (line) => /^ {4}Reviewed-By: ([^<]+)/.exec(line)[1].trim() ); async function getCollaboratorsFromReadme() { @@ -72,7 +72,7 @@ async function getCollaboratorsFromReadme() { foundCollaboratorHeading = true; } if (line.startsWith(' **') && isCollaborator) { - const [, name, email] = /^ \*\*([^*]+)\*\* <<(.+)>>/.exec(line); + const [, name, email] = /^ {2}\*\*([^*]+)\*\* <<(.+)>>/.exec(line); const mailmap = await runGitCommand( `git check-mailmap '${name} <${email}>'` ); @@ -136,7 +136,7 @@ async function moveCollaboratorToEmeritus(peopleToMove) { if (line.startsWith('* ')) { collaboratorFirstLine = line; } else if (line.startsWith(' **')) { - const [, name, email] = /^ \*\*([^*]+)\*\* <<(.+)>>/.exec(line); + const [, name, email] = /^ {2}\*\*([^*]+)\*\* <<(.+)>>/.exec(line); if (peopleToMove.some((entry) => { return entry.name === name && entry.email === email; })) {