-
Notifications
You must be signed in to change notification settings - Fork 10
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
Implement file-level parallelization #100
Conversation
Codecov Report
@@ Coverage Diff @@
## main #100 +/- ##
==========================================
- Coverage 95.63% 95.54% -0.10%
==========================================
Files 60 60
Lines 2451 2446 -5
==========================================
- Hits 2344 2337 -7
- Misses 107 109 +2
|
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.
How much of a PITA would it be to add an integration test with --max-workers used?
e49878a
to
15716b7
Compare
@clavedeluna since the default is to use one worker and since this is purely to enable performance testing I'm going to skip the integration test for now. Once we decide we want to use |
15716b7
to
43b88ab
Compare
Overview
Implement (optional) per-codemod file-level parallelization
Description
FileContext
and then aggregated at the end of each codemod pass