fix: change 'fetch' script by replacing wildcard with explicit fetch …#7804
fix: change 'fetch' script by replacing wildcard with explicit fetch …#7804TusharThakur04 wants to merge 1 commit intowebpack:mainfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
@alexander-akait can you please review? |
| "content": "node src/scripts/build-content-tree.mjs ./src/content ./src/_content.json", | ||
| "bundle-analyze": "run-s clean fetch content && webpack --config webpack.prod.mjs --config-node-env production && run-s printable content && webpack --config webpack.ssg.mjs --config-node-env production --env ssg --profile --json > stats.json && webpack-bundle-analyzer stats.json", | ||
| "fetch-repos": "node src/utilities/fetch-package-repos.mjs", | ||
| "fetch": "run-p fetch:*", |
There was a problem hiding this comment.
Weird, it should work, maybe we should use ** here?
There was a problem hiding this comment.
Do you install node_modules?
There was a problem hiding this comment.
yeah i do
could yarn version be the case?
i ve 4.12.0
There was a problem hiding this comment.
Very strange, works fine locally
There was a problem hiding this comment.
yes same here. Could you provide OS info and also have you runned yarn install / yarn first?
There was a problem hiding this comment.
yeah i ve done the setup using yarn
info:
or anything specific you want
There was a problem hiding this comment.
No, it should work, very strange, maybe you have global installed some packages?
3 participants
fixes #7803
Summary
This PR changes
fetchscript by replacing wildcard with explicit fetch scriptsWhat kind of change does this PR introduce?
fix
Did you add tests for your changes?
no
Does this PR introduce a breaking change?
no
If relevant, what needs to be documented once your changes are merged or what have you already documented?