-
Notifications
You must be signed in to change notification settings - Fork 9.3k
add Puppeteer.executablePath() method #771
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
add Puppeteer.executablePath() method #771
Conversation
This patch adds Puppeteer.executablePath() method to query the path of bundled chromium. Fixes puppeteer#745
docs/api.md
Outdated
| This methods attaches Puppeteer to an existing Chromium instance. | ||
|
|
||
| #### puppeteer.executablePath() | ||
| - returns: <[string]> A path where Puppeteer expects to find bundled chromium. Chromium might not exist there if the download was skipped with `PUPPETEER_SKIP_CHROMIUM_DOWNLOAD`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
link to the env vars section
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/chromium/Chromium
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
|
|
||
| This methods attaches Puppeteer to an existing Chromium instance. | ||
|
|
||
| #### puppeteer.executablePath() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This header is missing in the TOC.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good as long as the other comments are addressed.
|
I added an alternative solution for #745 |
|
@Janpot the |
This patch adds Puppeteer.executablePath() method to query the path
of bundled chromium.
Fixes #745