Skip to content
This repository has been archived by the owner on Mar 21, 2022. It is now read-only.

Relocate remaining packages that are included in shaded jar #1111

Merged
merged 1 commit into from
Jan 15, 2019

Conversation

DavidHaunsch
Copy link
Contributor

As mentioned in #1096 (comment), I'm trying here to relocate all remaining packages that are in the shaded jar but not relocated yet.
This will solves problems where a docker-client doesn't get the version from inside the shaded jar but the one that appears first on classpath.

@DavidHaunsch DavidHaunsch changed the title * relocate remaining packages that are included in shaded jar Relocate remaining packages that are included in shaded jar Nov 12, 2018
@codecov
Copy link

codecov bot commented Nov 12, 2018

Codecov Report

Merging #1111 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1111   +/-   ##
=========================================
  Coverage     70.17%   70.17%           
  Complexity      791      791           
=========================================
  Files           178      178           
  Lines          3772     3772           
  Branches        393      393           
=========================================
  Hits           2647     2647           
  Misses          960      960           
  Partials        165      165

@DavidHaunsch
Copy link
Contributor Author

Any chance this PR get's merged?
@davidxia (just to mention one who approved a PR lately) or someone else?

Copy link
Contributor

@davidxia davidxia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems good to me. @mattnworb, are you OK with this?

@davidxia davidxia merged commit d6e17c2 into spotify:master Jan 15, 2019
@DavidHaunsch
Copy link
Contributor Author

Nice, thanks!
Are there any timeplans for the next release?

@davidxia
Copy link
Contributor

@DavidHaunsch just released as 8.15.0

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants