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(docs): mention puppeteer@chrome tagged releases #3528

Merged
merged 3 commits into from Jan 11, 2019

Conversation

JoelEinbinder
Copy link
Collaborator

We should also add a note about them to the [For Project Maintainers] section of CONTRIBUTING.md, but I am not sure what the process actually is.

README.md Outdated
npm i puppeteer-core
# or "yarn add puppeteer-core"
npm i puppeteer-core@chrome-71
# or "yarn add puppeteer-core@chrome-71"
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd keep this as, otherwise it is confusing: reader at this point has no idea what's chrome-71 and why they should install this version of pptr-core.

README.md Outdated
@@ -312,6 +313,11 @@ This is not an artificial constraint: A lot of work on Puppeteer is actually tak

However, oftentimes it is desirable to use Puppeteer with the official Google Chrome rather than Chromium. For this to work, you should pick the version of Puppeteer that uses the Chromium version close enough to Chrome.

Since Chrome 70, the latest supported version Puppeteer is tagged on npm. To install Puppeteer for use with Chrome 71:
Copy link
Contributor

Choose a reason for hiding this comment

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

the latest supported version Puppeteer is tagged on npm

I am not sure, but is some word missing here?

@satanninja
Copy link

close

@aslushnikov
Copy link
Contributor

@JoelEinbinder updated this - ptal

Copy link
Collaborator Author

@JoelEinbinder JoelEinbinder left a comment

Choose a reason for hiding this comment

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

lgtm

@aslushnikov aslushnikov merged commit c44564a into puppeteer:master Jan 11, 2019
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

4 participants