tools: use GITHUB_ACTIONS env var in inactivity scripts

Don't rewrite the README if we're not running inside a GitHub Action. PR-URL: #41422 Reviewed-By: Anatoli Papirovski <[email protected]> Reviewed-By: Michaël Zasso <[email protected]>
nodejs · Jan 10, 2022 · c8d38e4 · c8d38e4
1 parent f65bbce
commit c8d38e4
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 10 deletions.
diff --git a/tools/find-inactive-collaborators.mjs b/tools/find-inactive-collaborators.mjs
@@ -189,8 +189,9 @@ const inactive = collaborators.filter((collaborator) =>
 if (inactive.length) {
   console.log('\nInactive collaborators:\n');
   console.log(inactive.map((entry) => `* ${entry.name}`).join('\n'));
-  console.log('\nGenerating new README.md file...');
-  const newReadmeText = await moveCollaboratorToEmeritus(inactive);
-  fs.writeFileSync(new URL('../README.md', import.meta.url), newReadmeText);
-  console.log('Updated README.md generated. Please commit these changes.');
+  if (process.env.GITHUB_ACTIONS) {
+    console.log('\nGenerating new README.md file...');
+    const newReadmeText = await moveCollaboratorToEmeritus(inactive);
+    fs.writeFileSync(new URL('../README.md', import.meta.url), newReadmeText);
+  }
 }
diff --git a/tools/find-inactive-tsc.mjs b/tools/find-inactive-tsc.mjs
@@ -260,10 +260,11 @@ if (inactive.length) {
   });
   console.log(`DETAILS_FOR_COMMIT_BODY=${commitDetails.join(' ')}`);
 
-  // Using console.warn() to avoid messing with find-inactive-tsc which consumes
-  // stdout.
-  console.warn('Generating new README.md file...');
-  const newReadmeText = await moveTscToEmeritus(inactive);
-  fs.writeFileSync(new URL('../README.md', import.meta.url), newReadmeText);
-  console.warn('Updated README.md generated. Please commit these changes.');
+  if (process.env.GITHUB_ACTIONS) {
+    // Using console.warn() to avoid messing with find-inactive-tsc which
+    // consumes stdout.
+    console.warn('Generating new README.md file...');
+    const newReadmeText = await moveTscToEmeritus(inactive);
+    fs.writeFileSync(new URL('../README.md', import.meta.url), newReadmeText);
+  }
 }