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

Remove uses of 'provided' in the eureka client #1293

Merged
merged 2 commits into from
May 11, 2020

Conversation

drobertduke
Copy link
Contributor

After updating gradle, we can stop using the deprecated 'provided' configuration.
This fixes an issue where the jackson-dataformat-xml and woodstox-core dependencies are affecting projects consuming eureka-client.

After updating gradle, we can stop using the deprecated 'provided' configuration.
This fixes an issue where the jackson-dataformat-xml and woodstox-core dependencies are affecting projects consuming eureka-client.
@troshko111
Copy link
Contributor

What about the providedCompile (2 hits), should those go to?

@kilink
Copy link
Member

kilink commented May 8, 2020

Nitpick, but maybe the wording of this comment should be updated as well:

// These dependencies are marked 'provided' in the client, but we need them always on the server

@drobertduke
Copy link
Contributor Author

What about the providedCompile (2 hits), should those go to?

providedCompile seems to be still used to exclude dependencies from War files gradle/gradle#1171

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