Skip to content

Commit

Permalink
Add CLI params to turn off LCP/LS screenshots.
Browse files Browse the repository at this point in the history
They already exists in Browsertime.

#4189
  • Loading branch information
soulgalore committed Jun 25, 2024
1 parent b37f342 commit 9703296
Showing 1 changed file with 18 additions and 0 deletions.
18 changes: 18 additions & 0 deletions lib/cli/cli.js
Original file line number Diff line number Diff line change
Expand Up @@ -1466,6 +1466,22 @@ export async function parseCommandLine() {
})

/** Screenshot */
.option('browsertime.screenshotLCP', {
alias: 'screenshotLCP',
type: 'boolean',
default: false,
describe:
'Save one screenshot per iteration that shows the largest contentful paint element (if the browser supports LCP).',
group: 'Screenshot'
})
.option('browsertime.screenshotLS', {
alias: 'screenshotLS',
type: 'boolean',
default: false,
describe:
'Save one screenshot per iteration that shows the layout shift elements (if the browser supports layout shift).',
group: 'Screenshot'
})
.option('browsertime.screenshot', {
type: 'boolean',
describe: 'Set to false to disable screenshots',
Expand Down Expand Up @@ -2042,6 +2058,8 @@ export async function parseCommandLine() {
.alias('help', 'h')
.config(config)
.hide('disableAPI')
.hide('browsertime.screenshotLCP')
.hide('browsertime.screenshotLS')
.alias('version', 'V')
.version(version)
.coerce('budget', function (argument) {
Expand Down

0 comments on commit 9703296

Please sign in to comment.