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
fix on android #296
fix on android #296
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch! Thanks 🎉
@@ -3,7 +3,7 @@ import * as os from 'os'; | |||
|
|||
import { FileSystemAdapter, Pattern } from './types'; | |||
|
|||
const CPU_COUNT = os.cpus().length; | |||
const CPU_COUNT = Math.max(os.cpus().length, 1); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add a comment to the code above that this is a specific hack for android?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
on Android, os.cups().length equals 0
👍 I was going to send over this same change -- |
Thanks for the contributing 🎉 |
@mrmlnc Would it be possible to cut a new release as this fixes issues with a number of downstream projects, and I'm sure they'd all love a stable version to point to rather than pointing to master. Thanks! |
I have setup the nightly builds for |
What is the purpose of this pull request?
fix on android
What changes did you make? (Give an overview)
fix
concurrency
minimal value