Skip to content

Commit 60ac1e6

Browse files
PackageToJS: Remove "playwright" dependency declaration from generated package.json
We alreawdy recommend users to declare playwright dep in their own package.json, so it doesn't make sense to have it declared here as well. ``` Playwright is not available in the current environment. Please run the following command to install it: $ npm install playwright && npx playwright install chromium ``` Actually, the declaration also makes things more complicated when users already have playwright dependency declared with a different version. Given that users unlikely commit the generated package.json into their repositories, it can't be a part of "workspace" either, so there is no benefit to have it here.
1 parent c6353c4 commit 60ac1e6

File tree

1 file changed

+0
-8
lines changed

1 file changed

+0
-8
lines changed

Plugins/PackageToJS/Templates/package.json

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -9,13 +9,5 @@
99
},
1010
"dependencies": {
1111
"@bjorn3/browser_wasi_shim": "0.3.0"
12-
},
13-
"peerDependencies": {
14-
"playwright": "^1.51.0"
15-
},
16-
"peerDependenciesMeta": {
17-
"playwright": {
18-
"optional": true
19-
}
2012
}
2113
}

0 commit comments

Comments
 (0)