diff --git a/utils/doclint/cli.js b/utils/doclint/cli.js index bbcf01fb28..0d1bbe7f35 100755 --- a/utils/doclint/cli.js +++ b/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} */ 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()}`)); + messages.push(...preprocessor.autocorrectInvalidLinks(PROJECT_DIR, mdSources, getRepositoryFiles())); - 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.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() { diff --git a/utils/doclint/preprocessor/index.js b/utils/doclint/preprocessor/index.js index edfa1e29c6..529e284ff0 100644 --- a/utils/doclint/preprocessor/index.js +++ b/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') diff --git a/utils/roll_browser.js b/utils/roll_browser.js index 4b09a04818..bcb6e23ee6 100755 --- a/utils/roll_browser.js +++ b/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) { }