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

docs(contributing): remove npm dist tags requirement #6571

Merged
merged 1 commit into from
Nov 4, 2020
Merged

Conversation

mathiasbynens
Copy link
Member

The npm dist tags are not worth the hassle, and are arguably harmful to the Web. Developers should feel encouraged to always test the latest available Puppeteer & Chromium versions, as opposed to any specific version.

Issue: #6482

@jackfranklin
Copy link
Collaborator

Looks like you might have to run npm run doc to satisfy the doc linting check (not 100% sure why but that's partly why CI is red).

@jackfranklin
Copy link
Collaborator

Also blocked on #6574, once that is in we can rebase and CI should be happy.

@mathiasbynens
Copy link
Member Author

Looks like you might have to run npm run doc to satisfy the doc linting check (not 100% sure why but that's partly why CI is red).

Done (it wanted to update the ToC)

The npm dist tags are not worth the hassle, and are arguably harmful to the Web. Developers should feel encouraged to always test the latest available Puppeteer & Chromium versions, as opposed to any specific version.

Issue: #6482
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants