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

Upgrade maven-shade-plugin from 2.4.1 to 3.1.0 #922

Merged
merged 1 commit into from
Nov 15, 2017

Conversation

davidxia
Copy link
Contributor

@davidxia davidxia commented Nov 13, 2017

and use ServicesResourceTransformer to relocate class names in
META-INF/services/.

fixes #900

@codecov-io
Copy link

codecov-io commented Nov 13, 2017

Codecov Report

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

@@            Coverage Diff            @@
##             master     #922   +/-   ##
=========================================
  Coverage     66.04%   66.04%           
  Complexity      769      769           
=========================================
  Files           171      171           
  Lines          3240     3240           
  Branches        369      369           
=========================================
  Hits           2140     2140           
  Misses          940      940           
  Partials        160      160

and use ServicesResourceTransformer to relocate class names in
META-INF/services/.

fixes #900
Copy link
Contributor

@caipre caipre left a comment

Choose a reason for hiding this comment

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

If they agree that this resolves their issue.

Copy link
Member

@mattnworb mattnworb left a comment

Choose a reason for hiding this comment

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

LGTM, I've seen ServicesResourceTransformer used for things like this in the past too 👍

@davidxia davidxia merged commit e04be81 into master Nov 15, 2017
@davidxia davidxia deleted the dxia/upgrade-shade-plugin branch November 15, 2017 16:59
@bjcohen
Copy link

bjcohen commented Nov 15, 2017

Sorry, just seeing this -- but looks good from what I can tell.

@mhagnumdw
Copy link

Sorry for the delay. Problem solved.

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.

version 8.9.1 (and 8.9.2-SNAPSHOT) shaded jar not working
6 participants