-
Notifications
You must be signed in to change notification settings - Fork 97
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
npm run benchamark
doesn't work in local
#313
Comments
I found this issue while looking at the GFI 🐤 but it works well for me. |
both of my laptops use windows10 without WSL and the problem arises.
( |
This issue seems to be caused by using the glob pattern as argv of ts-node. I think it will be resolved by setting only one path as the entry point without using the glob pattern. CauseAfter testing, some console environments did not support the glob pattern. case 1 (macOS, zsh) - is working
[
'/usr/local/bin/node',
'/Users/user/Desktop/yorkie-js-sdk/node_modules/.bin/ts-node-script',
'test/bench/document_test.ts'
] case 2 (Windows, cmd) - is not working
[
'C:\\Program Files\\nodejs\\node.exe',
'C:\\Users\\user\\Desktop\\test\\node_modules\\ts-node\\dist\\bin.js',
'./**.ts' # 🚨 do not support glob
] Reference
Could I try this issue? @mojosoeun @blurfx |
@taenykim Sure :) |
What happened:
npm run benchamark
doesn't work in local (verified in Windows and Mac)What you expected to happen:
I expected it works normally like in CI but it returns error.
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
The other commands that run ts-node running a single script work normally.
If someone needs to test all bench in local, try
./node_modules/ts-node/dist/bin-script.js ./test/bench/**.ts
instead ofnpm run benchmark
.Environment:
yorkie version
):The text was updated successfully, but these errors were encountered: