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

[2.x] feat(parallel): Adds support for plugins to filter parallel arguments #703

Merged
merged 7 commits into from Mar 24, 2023

Conversation

lukeraymonddowning
Copy link
Member

Q A
Bug fix? yes
New feature? yes
Fixed tickets #682

Basically, CallsHandlesArguments is not executed fully in parallel (and rightly so). However, this means that plugins have no way of altering arguments and thus supporting parallel execution.

This PR adds a new action and contract that allows plugins to alter parallel arguments before parallel execution.

@owenvoke
Copy link
Member

@lukeraymonddowning, I still don't think that this resolves the issue... 🤔 Unless I'm implementing the contract wrongly, it seems to have stripped the --gql-coverage flag before it even gets to the plugin.

@nunomaduro nunomaduro merged commit 33f596b into 2.x Mar 24, 2023
28 checks passed
@nunomaduro nunomaduro deleted the 2.x_parallel_args branch March 24, 2023 11:22
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.

None yet

3 participants