Skip to content
This repository was archived by the owner on Jun 30, 2023. It is now read-only.

Replaced printStackTrace() with JUL#150

Merged
tangiel merged 1 commit intocloudendpoints:masterfrom
nbali:patch-2
Jul 18, 2018
Merged

Replaced printStackTrace() with JUL#150
tangiel merged 1 commit intocloudendpoints:masterfrom
nbali:patch-2

Conversation

@nbali
Copy link
Contributor

@nbali nbali commented Jul 17, 2018

If an enum was misspelled, this printStrackTrace spammed the logs with WARNING lines.

@codecov-io
Copy link

codecov-io commented Jul 17, 2018

Codecov Report

Merging #150 into master will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #150   +/-   ##
=========================================
  Coverage     80.01%   80.01%           
  Complexity     1682     1682           
=========================================
  Files           156      156           
  Lines          5599     5599           
  Branches        731      731           
=========================================
  Hits           4480     4480           
  Misses          840      840           
  Partials        279      279
Impacted Files Coverage Δ Complexity Δ
...ver/spi/request/RestServletRequestParamReader.java 93.44% <0%> (ø) 23 <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 bc1d80a...f4c0b98. Read the comment docs.

@nbali
Copy link
Contributor Author

nbali commented Jul 17, 2018

@tangiel ... and if if it's okay, I would like to get it released, thx :)

@tangiel
Copy link
Contributor

tangiel commented Jul 18, 2018

Thanks. I will release it in 2.1.1.

@tangiel tangiel merged commit 537a48b into cloudendpoints:master Jul 18, 2018
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.

3 participants