Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add warn for headless: true #10039

Merged
merged 4 commits into from Apr 21, 2023
Merged

Conversation

Lightning00Blade
Copy link
Collaborator

@Lightning00Blade Lightning00Blade commented Apr 18, 2023

Warn for future behavior change of puppeteer.launch({headless: true})

The message can be suppressed by setting:
Env - PUPPETEER_DISABLE_HEADLESS_WARNING=true, (or via general logging PUPPETEER_LOGLEVEL=silent, PUPPETEER_LOGLEVEL=error,)
Config - logLevel: 'silent', logLevel: 'error'
NPM LogLevel - https://docs.npmjs.com/cli/v9/using-npm/logging (silent or error)

Warning:
image

@Lightning00Blade Lightning00Blade force-pushed the depracate-old-headless branch 2 times, most recently from 6134889 to 1bf5732 Compare April 19, 2023 08:59
@Lightning00Blade Lightning00Blade marked this pull request as ready for review April 19, 2023 08:59
Copy link
Member

@mathiasbynens mathiasbynens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left some comments with suggestions

packages/puppeteer-core/src/node/ChromeLauncher.ts Outdated Show resolved Hide resolved
packages/puppeteer-core/src/node/LaunchOptions.ts Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@OrKoN
Copy link
Collaborator

OrKoN commented Apr 20, 2023

let's mark it as a feat to make it show up in the changelog

@Lightning00Blade Lightning00Blade changed the title chore: add warn for headless: true feat: add warn for headless: true Apr 20, 2023
@Lightning00Blade Lightning00Blade force-pushed the depracate-old-headless branch 2 times, most recently from 58df0d1 to 8643fc6 Compare April 20, 2023 14:00
Lightning00Blade and others added 4 commits April 21, 2023 12:57
DEPRECATION: `puppeteer.launch({headless: true})`
Co-authored-by: Mathias Bynens <mathias@qiwi.be>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants