Open lionelhorn opened 1 year ago
Could you check if swapping the quotes as follows works?
"build:all": 'yarn run concurrently --success=all -r -m=1 "yarn workspaces-to-typescript-project-references" "yarn turbo run prepublish"'
(that just seems cleaner to my mind anyhow)
We could update all commands in package.json to use this style of quoting in the right hand side.
I'll then have to check whether there's any linting tools that update or complain about the JSON.
Could you check if swapping the quotes as follows works?
"build:all": 'yarn run concurrently --success=all -r -m=1 "yarn workspaces-to-typescript-project-references" "yarn turbo run prepublish"'
(that just seems cleaner to my mind anyhow)
We could update all commands in package.json to use this style of quoting in the right hand side.
I'll then have to check whether there's any linting tools that update or complain about the JSON.
Unfortunately JSON doesn't allow single quotes :/ so we can't do this for package.json
Preflight Checklist
What package is this bug report for?
rrweb
Version
from source
Expected Behavior
Able to follow the contributing guide. https://github.com/rrweb-io/rrweb#contribute-guide without errors.
Actual Behavior
Step 3. Run yarn dev in the root to get auto-building for all the sub-packages whenever you modify anything. => Fails with import error.
Steps to Reproduce
Do step 1 to 3.
System is windows. Packages are not build and /dist/ aren't present. Then the yarn-dev fails.
May be related to npm tasks and quotes on windows.
Replacing https://github.com/rrweb-io/rrweb/blob/a6ce7182ce59da8a565b12f7dfcb87eb10ec7ca4/package.json#L39 with
seems to work.
Similar but a bit different from https://github.com/rrweb-io/rrweb/issues/12 Similar to https://github.com/rrweb-io/rrweb/issues/1221
Testcase Gist URL
No response
Additional Information