-
-
Notifications
You must be signed in to change notification settings - Fork 240
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
feat: honor concurrency in withPreparedCharts #345
Conversation
a9c4f55
to
edfc18b
Compare
Signed-off-by: Viktor Oreshkin <imselfish@stek29.rocks>
edfc18b
to
822b7b2
Compare
@stek29 Please fix ci error. |
@yxxhero I don't see how test failure has anything to do with my proposed changes – it fails locally too for me, even when running tests on master. are you sure it's up to me to fix and it has anything to do with my changes? maybe tests are just flaky/broken for external contributors? |
|
Signed-off-by: Viktor Oreshkin <imselfish@stek29.rocks>
09b3b4e
to
a56303b
Compare
@yxxhero please approve running test, I've tried to implement a fix. |
@stek29 Thanks very much. |
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.
LGTM
see #341
this PR also adds --concurrency flag to
deps
command.