-
Notifications
You must be signed in to change notification settings - Fork 358
Clean up -x task exclusions in ci.yml #2913
Copy link
Copy link
Closed
Labels
good first issueThese are recommended starting points for newcomers looking to make their first contributions.These are recommended starting points for newcomers looking to make their first contributions.help wantedCommunity contributions are especially encouraged for these issues.Community contributions are especially encouraged for these issues.triagedIssues labeled as 'Triaged' have been reviewed and are deemed actionable.Issues labeled as 'Triaged' have been reviewed and are deemed actionable.
Description
At the moment, ci.yml contains many gradle task invocations with long exclusion lists like this:
build test -Dbuild.snapshot=false
-x integrationTest
-x spotlessCheck
-x checkstyleMain
-x checkstyleTest
-x spotbugsMain
This is because the jacoco plugin enforces explicit incoming dependencies from all test tasks. This will however also cause the execution of these "through the backdoor" if only a single test task is supposed to be executed.
It would be nice if a solution could be found to avoid that.
See also the conversation in this PR: #2861 (comment)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
good first issueThese are recommended starting points for newcomers looking to make their first contributions.These are recommended starting points for newcomers looking to make their first contributions.help wantedCommunity contributions are especially encouraged for these issues.Community contributions are especially encouraged for these issues.triagedIssues labeled as 'Triaged' have been reviewed and are deemed actionable.Issues labeled as 'Triaged' have been reviewed and are deemed actionable.