Commit 2f68f71
Remove most pnpm hoisting, fix phantom deps (#24096)
We've long had pnpm's public hoisting disabled, but it also privately
hoists everything into `node_modules/.pnpm/node_modules/` for
compatibility with packages that have undeclared dependencies. Let's
disable most of that private hoisting and see what turns up.
Changes of note:
* We still hoist a bunch of eslint plugins (and one prettier plugin) as
untangling that mess seems like enough work to put off for later.
* Only a few upstream packages have bugs that need working around:
* `@mdx-js/loader`: mdx-js/mdx#2019
* `@automattic/components`: Missing dep on `@wordpress/base-styles`.
And the next version will probably be unusable for us due to added
`i18n-calypso`.
* `@automattic/popup-monitor`: Missing dep on `events`.
* markdown-it`: Missing dep on `punycode`.
markdown-it/markdown-it#230
* `@samverschueren/stream-to-observable`: Outdated dep on `any-observable`.
SamVerschueren/stream-to-observable#9
Hacking around that should also fix p1649254510834369-slack-CBG1CP4EN.
* `git-node-fs`: Missing peer dep on `js-git`.
creationix/git-node-fs#8
* `fetch-mock`'s peer dep on `node-fetch` is optional, but only because
they also allow running in-browser. It's required for node.
* Added `webpack-cli` alongside `webpack` everywhere to make sure
p1650571211251179-slack-CBG1CP4EN is fixed. Dropped Webpack entirely
from Boost instead though, they don't use it.
* Had our webpack-config package point to its own copy of
`@babel/runtime` instead of making that a peer dep.
* Jetpack's extensions tests use a ton of `@wordpress/` packages that
weren't being directly depended on.
Committed via a GitHub action: https://github.com/Automattic/jetpack/actions/runs/22340093761 parent c3f56c4 commit 2f68f71
3 files changed
+18
-4
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
5 | 5 | | |
6 | 6 | | |
7 | 7 | | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
8 | 15 | | |
9 | 16 | | |
10 | 17 | | |
| |||
222 | 229 | | |
223 | 230 | | |
224 | 231 | | |
| 232 | + | |
225 | 233 | | |
226 | 234 | | |
227 | 235 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
13 | 13 | | |
14 | 14 | | |
15 | 15 | | |
| 16 | + | |
16 | 17 | | |
17 | 18 | | |
18 | | - | |
| 19 | + | |
| 20 | + | |
19 | 21 | | |
20 | 22 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
21 | 21 | | |
22 | 22 | | |
23 | 23 | | |
24 | | - | |
25 | | - | |
26 | | - | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
27 | 31 | | |
28 | 32 | | |
29 | 33 | | |
| |||
0 commit comments