-
Notifications
You must be signed in to change notification settings - Fork 211
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
(chore) Chain tasks in the verify script and lower concurrency #988
Conversation
Size Change: 0 B Total Size: 2.96 MB ℹ️ View Unchanged
|
Oh, this is awesome, running verify was really clogging resources, and it was almost impossible to push from with IntelliJ because the verify basically crashed IntelliJ, thanks @denniskigen |
If resource usage is still a concern, we could just remove the concurrency flag altogether. Let me know how things go, @mogoodrich. |
Removed, it defaults to |
Requirements
Summary
This PR chains tasks in the
verify
script and lowers the concurrency of the task execution so a smaller percentage of the available logical processors is used. This should make it less likely that executing the script will slow down your computer.Screenshots
Related Issue
Other