Skip to content

Make puppeteer a peerDependency - #54

Merged
marvinhagemeister merged 2 commits into
masterfrom
puppeteer-peerdep
Jul 7, 2020
Merged

Make puppeteer a peerDependency#54
marvinhagemeister merged 2 commits into
masterfrom
puppeteer-peerdep

Conversation

@marvinhagemeister

Copy link
Copy Markdown
Collaborator

This avoids duplicate chromium downloads if the user has a different version installed in his project.

Comment thread package.json Outdated
"simple-code-frame": "^1.0.0"
},
"peerDependencies": {
"puppeteer": "^4.0.1",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

We could probably relax this a bit and do something like ">=2" I think. May be able to do "*" since karma-chrome-launcher uses the chrome.exe directly I think and doesn't use the puppeteer API directly

@marvinhagemeister marvinhagemeister Jul 7, 2020

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

oh good point 👍 Updated the PR 🎉

@andrewiggins

Copy link
Copy Markdown
Collaborator

Reminder, this'll require a major semver update

@marvinhagemeister
marvinhagemeister merged commit b2f613a into master Jul 7, 2020
@marvinhagemeister
marvinhagemeister deleted the puppeteer-peerdep branch July 7, 2020 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants