Skip to content

Upgrade checkstyle version #1132

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 2 commits into from
Mar 19, 2019
Merged

Upgrade checkstyle version #1132

merged 2 commits into from
Mar 19, 2019

Conversation

zoewangg
Copy link
Contributor

@zoewangg zoewangg commented Mar 14, 2019

Description

Upgrade checkstyle version per https://nvd.nist.gov/vuln/detail/CVE-2019-9658

Hopefully it still builds

Had to remove a bunch of broken checkstyles.

License

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

@zoewangg zoewangg force-pushed the zoewang-checkstyleVersion branch 3 times, most recently from bca1b09 to 7daff95 Compare March 18, 2019 23:14
@zoewangg zoewangg force-pushed the zoewang-checkstyleVersion branch from 7daff95 to 185aa13 Compare March 18, 2019 23:23
@codecov-io
Copy link

codecov-io commented Mar 18, 2019

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1132   +/-   ##
=========================================
  Coverage     58.81%   58.81%           
  Complexity     4564     4564           
=========================================
  Files           742      742           
  Lines         22892    22892           
  Branches       1706     1706           
=========================================
  Hits          13465    13465           
  Misses         8734     8734           
  Partials        693      693
Impacted Files Coverage Δ Complexity Δ
...ttp/pipeline/stages/MakeAsyncHttpRequestStage.java 80.21% <ø> (ø) 14 <0> (ø) ⬇️
...ine/stages/ApiCallAttemptTimeoutTrackingStage.java 100% <ø> (ø) 2 <0> (ø) ⬇️
...p/pipeline/stages/ApiCallTimeoutTrackingStage.java 82.75% <ø> (ø) 9 <0> (ø) ⬇️

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 7428f62...5f7c8d1. Read the comment docs.

@zoewangg zoewangg merged commit ad2dfe8 into master Mar 19, 2019
@zoewangg zoewangg deleted the zoewang-checkstyleVersion branch April 3, 2019 03:35
aws-sdk-java-automation added a commit that referenced this pull request Dec 31, 2020
…f5c07ff95

Pull request: release <- staging/e1f69a7c-1276-4d03-9ce2-6f3f5c07ff95
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