Nodejs/node
1
0
Fork 0
mirror of https://github.com/nodejs/node.git synced 2024-11-21 10:59:27 +00:00
Code Issues Actions 25 Packages Projects Releases Wiki Activity

tools: use {N} for spaces in regex

Browse Source 
Spaces are hard to count. Use {N} notation to indicate how many spaces
in regular expressions in find-inactive-collaborators.

PR-URL: https://github.com/nodejs/node/pull/41295
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
This commit is contained in:
Rich Trott 2021-12-23 15:19:19 -08:00 committed by GitHub
parent 406e6d87c4
commit 2b63dfe423
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
tools/find-inactive-collaborators.mjs
View File

@ -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;
})) {