-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Prune some test worker dependencies #29217
Conversation
Can we move this forward? |
eb019ea
to
7cb355f
Compare
@bot-gradle test this |
I've triggered the following builds for you. Click here to see all build failures. |
The eventual goal is for ForkedTestClasspathFactory to not need a hard-coded list of gradle modules for worker classpaths. We should be able to generate that automatically based on the real dependencies of the test worker's runtime classpath. This commit begins to clean up the dependency graph of the test classpath, removing some dependencies on Gradle modules and external dependencies that don't necessarily need to be there. There is still quite some work to do, but this starts the process.
7cb355f
to
fc494af
Compare
@bot-gradle test this |
I've triggered the following builds for you. Click here to see all build failures. |
The merge queue build has failed. Click here to see all failures. |
sb.append(")"); | ||
} | ||
|
||
return sb.toString(); | ||
} | ||
|
||
// Can be replaced with String.join when workers are updated to run in JDK 8. | ||
private static String join(Set<String> strings, String delimiter) { |
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.
What about CollectionUtils.join(...)
?
The eventual goal is for ForkedTestClasspathFactory to not need a hard-coded list of gradle modules for worker classpaths. We should be able to generate that automatically based on the real dependencies of the test worker's runtime classpath.
This commit begins to clean up the dependency graph of the test classpath, removing some dependencies on Gradle modules and external dependencies that don't necessarily need to be there. There is still quite some work to do, but this starts the process.
Reviewing cheatsheet
Before merging the PR, comments starting with