Skip to content

Moved SDK dependencies from the root pom.xml to a bom-internal module. #1066

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

Merged
merged 3 commits into from
Feb 8, 2019

Conversation

millems
Copy link
Contributor

@millems millems commented Feb 7, 2019

This prevents dependencies in the root pom from polluting the bom module. Also added some missing bom dependencies and cleaned up the pom.xml files across the repository.

Fixes #1063 and #295.

@millems millems force-pushed the millem/fix-bom branch 6 times, most recently from 1f9a43a to 3c3c545 Compare February 7, 2019 18:52
@codecov-io
Copy link

codecov-io commented Feb 7, 2019

Codecov Report

Merging #1066 into master will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1066      +/-   ##
============================================
+ Coverage     56.18%   56.19%   +0.01%     
- Complexity     4672     4674       +2     
============================================
  Files           822      822              
  Lines         27972    27972              
  Branches       2253     2253              
============================================
+ Hits          15716    15720       +4     
+ Misses        11525    11523       -2     
+ Partials        731      729       -2
Impacted Files Coverage Δ Complexity Δ
...are/amazon/awssdk/core/internal/util/Mimetype.java 80.48% <0%> (+2.43%) 15% <0%> (+1%) ⬆️
...nio/netty/internal/OldConnectionReaperHandler.java 90.9% <0%> (+9.09%) 15% <0%> (+1%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1fafd9e...3b14759. Read the comment docs.

@millems millems force-pushed the millem/fix-bom branch 2 times, most recently from 309a521 to afeb390 Compare February 7, 2019 23:08
…. This prevents dependencies in the root pom from polluting the bom module. Also added missing core modules to the bom and cleaned up the pom.xml files across the repository.
Copy link
Contributor

@varunnvs92 varunnvs92 left a comment

Choose a reason for hiding this comment

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

Looks good.

@varunnvs92
Copy link
Contributor

Did you verify locally that the issues mentioned in the original issues get fixed by this change?

@millems
Copy link
Contributor Author

millems commented Feb 8, 2019

Verified!

@millems millems merged commit d4d8e5a into master Feb 8, 2019
@millems millems deleted the millem/fix-bom branch February 8, 2019 01:37
aws-sdk-java-automation added a commit that referenced this pull request Dec 2, 2020
…4116407bc

Pull request: release <- staging/82b79bb8-ba31-48fb-9b3a-b594116407bc
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.

3 participants