tumblr / colossus

I/O and Microservice library for Scala
Apache License 2.0
1.14k stars 96 forks source link

increase the parser limit so it always gets overriden by the configu… #664

Closed jlbelmonte closed 6 years ago

jlbelmonte commented 6 years ago

Solves https://github.com/tumblr/colossus/issues/656

The configuration limit will override this, so let's set it pretty high so it doesn't create issues.

@benblack86 @amotamed @aliyakamercan @DanSimon

codecov-io commented 6 years ago

Codecov Report

Merging #664 into develop will not change coverage. The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #664   +/-   ##
========================================
  Coverage    85.72%   85.72%           
========================================
  Files          105      105           
  Lines         4497     4497           
  Branches       426      426           
========================================
  Hits          3855     3855           
  Misses         642      642
Impacted Files Coverage Δ
...olossus/src/main/scala/colossus/util/Parsing.scala 90.88% <100%> (ø) :arrow_up:

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 debac07...53189dd. Read the comment docs.