Skip to content
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

[Core] Add $seconds, $maxNumberOfProcess, and $jobSize parameters to RectorConfig::parallel() method #2188

Merged
merged 5 commits into from
Apr 28, 2022

Conversation

samsonasik
Copy link
Member

to handle use Option directly to increase parallel timeout like this codeigniter4/CodeIgniter4@346c83f#diff-d933a2ad57daa43419b483d3f1ddedaec7aa44933007fead9f9437390f596f4a

@TomasVotruba
Copy link
Member

I was thinking about that, but it might be more easier to find this under parallel() method. We needed this on previous parameters approach, but it was not possible.

Could you move it to parallel() method with optional params? Same for the other parallel parameter

@samsonasik
Copy link
Member Author

I've moved to parallel() method.

@samsonasik samsonasik changed the title [Core] Add RectorConfig->parallelTimeoutInSeconds() method [Core] Add $seconds parameter to RectorConfig::parallel() method Apr 28, 2022
@samsonasik
Copy link
Member Author

All checks have passed 🎉 @TomasVotruba I think it is ready.

@samsonasik
Copy link
Member Author

I've added more params for parallel job size and parallel max number of process, place in 2nd and 3rd param as most of the time, the changed part is the $second parameter.

@samsonasik samsonasik changed the title [Core] Add $seconds parameter to RectorConfig::parallel() method [Core] Add $seconds, $maxNumberOfProcess, and $jobSize parameters to RectorConfig::parallel() method Apr 28, 2022
@TomasVotruba TomasVotruba merged commit 680e8a1 into main Apr 28, 2022
@TomasVotruba TomasVotruba deleted the add-parallel-timeout-in-seconds branch April 28, 2022 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants