Skip to content

Fix Gradle config for Eclipse IDE#2961

Merged
chanseokoh merged 2 commits intomasterfrom
fix-for-eclipse
Jan 5, 2021
Merged

Fix Gradle config for Eclipse IDE#2961
chanseokoh merged 2 commits intomasterfrom
fix-for-eclipse

Conversation

@chanseokoh
Copy link
Member

@loosebazooka
Copy link
Member

This seems kinda similar to https://github.com/GoogleContainerTools/jib/pull/2958/files, are the integration tests marked as "test" modules in this mechanism?

@chanseokoh
Copy link
Member Author

Yeah, src/integration-test/{java,resources} are still marked as "test" in my Eclipse after this change.

This PR resolves different issues than what #2958 used to do; it's unrelated to marking them "test." Without eclipse.classpath.plusConfigurations, Eclipse cannot resolve classes defined in integrationTestImplementation.

@chanseokoh chanseokoh merged commit 7517170 into master Jan 5, 2021
@chanseokoh chanseokoh deleted the fix-for-eclipse branch January 5, 2021 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants