Skip to content

fix classifier added by 'shadow' configuration#1200

Merged
littleaj merged 1 commit into
masterfrom
littleaj/fix-pom
Feb 13, 2020
Merged

fix classifier added by 'shadow' configuration#1200
littleaj merged 1 commit into
masterfrom
littleaj/fix-pom

Conversation

@littleaj

@littleaj littleaj commented Feb 12, 2020

Copy link
Copy Markdown
Contributor

Fix #1199.

Introduced by this change: c898d49
Apparently the 'shadow' configuration (defined by the shadowJar plugin) adds the 'all' classifier to the dependency in the pom by default.

Using the shadow configuration as a dependency simplifies the build script and prevents the need to change the spring-boot-starter build file if web or core dependencies change.

For significant contributions please make sure you have completed the following items:

  • Design discussion issue #
  • Changes in public surface reviewed
  • CHANGELOG.md updated

@littleaj littleaj added this to the 2.6.0-BETA.3 milestone Feb 12, 2020
@littleaj littleaj requested a review from a team February 12, 2020 02:33
@littleaj littleaj self-assigned this Feb 12, 2020
@littleaj littleaj requested review from trask and removed request for a team February 12, 2020 02:33

@trask trask left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for catching the issue with #1197!

@littleaj littleaj merged commit e9119f2 into master Feb 13, 2020
@littleaj littleaj deleted the littleaj/fix-pom branch February 13, 2020 01:44
trask pushed a commit that referenced this pull request Oct 17, 2020
* Implement MDC auto-instrumentation for log4j2

* Implement MDC auto-instrumentation for log4j2 2.7

* Implement MDC auto-instrumentation for log4j2

* Implement MDC auto-instrumentation for log4j2

* Implement MDC auto-instrumentation for log4j2

* Implement MDC auto-instrumentation for log4j1

* Implement MDC auto-instrumentation for log4j2
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.

Error: Could not resolve dependencies for latest snapshot

2 participants