Skip to content

Maven Surefire Plugin 2.22.2 #773

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 5 commits into from
Jan 9, 2020
Merged

Conversation

sullis
Copy link
Contributor

@sullis sullis commented Oct 20, 2018

Description

Maven Surefire Plugin 2.22.1

Notes: https://blogs.apache.org/maven/entry/apache-maven-surefire-plugin-version1

Motivation and Context

latest version

Testing

Travis Build

Screenshots (if appropriate)

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Checklist

  • I have read the CONTRIBUTING document
  • Local run of mvn install succeeds
  • My code follows the code style of this project
  • My change requires a change to the Javadoc documentation
  • I have updated the Javadoc documentation accordingly
  • I have read the README document
  • I have added tests to cover my changes
  • All new and existing tests passed
  • A short description of the change has been added to the CHANGELOG

License

  • I confirm that this pull request can be released under the Apache 2 license

@sullis sullis force-pushed the maven-surefire-2.22.1 branch from 6d0e1c6 to b2ca447 Compare October 20, 2018 19:09
Copy link
Contributor

@zoewangg zoewangg left a comment

Choose a reason for hiding this comment

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

Hmm, I don't think we need a changelog entry for testing dependency version bump. Could you remove it? I'll merge it afterwards. Thank you for the PR!

@sullis sullis force-pushed the maven-surefire-2.22.1 branch from e3c5691 to 2dc76c0 Compare November 18, 2018 21:49
@sullis
Copy link
Contributor Author

sullis commented Nov 18, 2018

Ready for review

@sullis sullis force-pushed the maven-surefire-2.22.1 branch from 2dc76c0 to e44fecd Compare January 12, 2019 18:55
@codecov-io
Copy link

codecov-io commented Jan 12, 2019

Codecov Report

Merging #773 into master will increase coverage by 4.28%.
The diff coverage is 89.51%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #773      +/-   ##
============================================
+ Coverage     71.25%   75.54%   +4.28%     
- Complexity      188      638     +450     
============================================
  Files           781      896     +115     
  Lines         24273    28064    +3791     
  Branches       1819     2214     +395     
============================================
+ Hits          17296    21200    +3904     
+ Misses         6180     5846     -334     
- Partials        797     1018     +221
Flag Coverage Δ Complexity Δ
#unittests 75.54% <89.51%> (+4.28%) 638 <0> (+450) ⬆️
Impacted Files Coverage Δ Complexity Δ
...egen/model/config/customization/ShapeModifier.java 100% <ø> (+31.25%) 0 <0> (ø) ⬇️
...sdk/codegen/model/service/HostPrefixProcessor.java 100% <ø> (ø) 0 <0> (ø) ⬇️
...k/codegen/poet/builder/BaseClientBuilderClass.java 79.25% <ø> (ø) 0 <0> (ø) ⬇️
...wssdk/codegen/lite/emitters/JavaCodeFormatter.java 84.84% <ø> (ø) 0 <0> (ø) ⬇️
...zon/awssdk/codegen/emitters/JavaCodeFormatter.java 84.84% <ø> (ø) 0 <0> (ø) ⬇️
...awssdk/codegen/emitters/GeneratorPathProvider.java 41.66% <ø> (ø) 0 <0> (ø) ⬇️
.../software/amazon/awssdk/codegen/CodeGenerator.java 0% <ø> (ø) 0 <0> (ø) ⬇️
...tware/amazon/awssdk/codegen/poet/StaticImport.java 60% <ø> (ø) 0 <0> (ø) ⬇️
...azon/awssdk/codegen/lite/emitters/LinkRemover.java 0% <ø> (ø) 0 <0> (ø) ⬇️
...mazon/awssdk/codegen/lite/emitters/CodeWriter.java 0% <ø> (ø) 0 <0> (ø) ⬇️
... and 435 more

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 fde0ef4...8bb0683. Read the comment docs.

@sullis
Copy link
Contributor Author

sullis commented Jan 12, 2019

ready for merge @zoewangg

@sullis sullis force-pushed the maven-surefire-2.22.1 branch from 5d95341 to f8201a2 Compare January 17, 2019 03:29
@sullis
Copy link
Contributor Author

sullis commented Jan 17, 2019

ready for merge

@sullis sullis force-pushed the maven-surefire-2.22.1 branch from 39b13f1 to 998468b Compare May 28, 2019 07:15
@sullis sullis changed the title Maven Surefire Plugin 2.22.1 Maven Surefire Plugin 2.22.2 May 28, 2019
@sullis sullis force-pushed the maven-surefire-2.22.1 branch from 998468b to aeeac41 Compare May 28, 2019 07:17
@dagnir
Copy link
Contributor

dagnir commented Jul 12, 2019

@sullis Could you please fix the conflict? I can merge after that.

@sullis sullis force-pushed the maven-surefire-2.22.1 branch from aeeac41 to 8ae9dc1 Compare July 14, 2019 01:52
@sullis sullis force-pushed the maven-surefire-2.22.1 branch from 8ae9dc1 to 75d1d7c Compare July 14, 2019 01:55
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 9, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@dagnir dagnir merged commit 2d9b61f into aws:master Jan 9, 2020
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.

4 participants