chore: remove the --only-update-browsers option

Author: pavelfeldman

utils/doclint/cli.js

@@ -39,7 +39,6 @@ run().catch(e => {
 
 async function run() {
   const startTime = Date.now();
-  const onlyBrowserVersions = process.argv.includes('--only-browser-versions');
 
   /** @type {!Array<!Message>} */
   const messages = [];
@@ -141,21 +140,19 @@ async function run() {
       libversion: VERSION,
       chromiumVersion: browserVersions.chromium,
       firefoxVersion: browserVersions.firefox,
-      onlyBrowserVersions,
     })));
 
-    if (!onlyBrowserVersions) {
-      messages.push(...preprocessor.autocorrectInvalidLinks(PROJECT_DIR, mdSources, getRepositoryFiles()));
-      for (const source of mdSources.filter(source => source.hasUpdatedText()))
-        messages.push(Message.warning(`WARN: updated ${source.projectPath()}`));
-
-      const browser = await playwright.chromium.launch();
-      const page = await browser.newPage();
-      const checkPublicAPI = require('./check_public_api');
-      const jsSources = await Source.readdir(path.join(PROJECT_DIR, 'src', 'client'), '', []);
-      messages.push(...await checkPublicAPI(page, [api], jsSources));
-      await browser.close();
-    }
+    messages.push(...preprocessor.autocorrectInvalidLinks(PROJECT_DIR, mdSources, getRepositoryFiles()));
+
+    for (const source of mdSources.filter(source => source.hasUpdatedText()))
+      messages.push(Message.warning(`WARN: updated ${source.projectPath()}`));
+
+    const browser = await playwright.chromium.launch();
+    const page = await browser.newPage();
+    const checkPublicAPI = require('./check_public_api');
+    const jsSources = await Source.readdir(path.join(PROJECT_DIR, 'src', 'client'), '', []);
+    messages.push(...await checkPublicAPI(page, [api], jsSources));
+    await browser.close();
 
     for (const source of mdSources) {
       if (!source.hasUpdatedText())
@@ -193,7 +190,7 @@ async function run() {
   console.log(`${errors.length} failures, ${warnings.length} warnings.`);
   const runningTime = Date.now() - startTime;
   console.log(`DocLint Finished in ${runningTime / 1000} seconds`);
-  process.exit(clearExit || onlyBrowserVersions ? 0 : 1);
+  process.exit(clearExit ? 0 : 1);
 }
 
 async function getBrowserVersions() {

utils/doclint/preprocessor/index.js

@@ -17,7 +17,7 @@
 const path = require('path');
 const Message = require('../Message');
 
-function runCommands(sources, {libversion, chromiumVersion, firefoxVersion, onlyBrowserVersions}) {
+function runCommands(sources, {libversion, chromiumVersion, firefoxVersion}) {
   // Release version is everything that doesn't include "-".
   const isReleaseVersion = !libversion.includes('-');
 
@@ -51,8 +51,6 @@ function runCommands(sources, {libversion, chromiumVersion, firefoxVersion, only
         newText = `[![Chromium version](https://img.shields.io/badge/chromium-${chromiumVersion}-blue.svg?logo=google-chrome)](https://www.chromium.org/Home)`;
       else if (commandName === 'firefox-version-badge')
         newText = `[![Firefox version](https://img.shields.io/badge/firefox-${firefoxVersion}-blue.svg?logo=mozilla-firefox)](https://www.mozilla.org/en-US/firefox/new/)`;
-      else if (onlyBrowserVersions)
-        continue;
       else if (commandName === 'version')
         newText = isReleaseVersion ? 'v' + libversion : 'Tip-Of-Tree';
       else if (commandName === 'toc')

utils/roll_browser.js

@@ -82,7 +82,7 @@ Example:
   // 5. Update docs.
   console.log('\nUpdating documentation...');
   try {
-    process.stdout.write(execSync('npm run --silent doc -- --only-browser-versions'));
+    process.stdout.write(execSync('npm run --silent doc'));
   } catch (e) {
   }