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

finatra/inject - Rename InjectUtils to more specific PoolUtils #258

Merged
merged 1 commit into from Sep 28, 2015

Conversation

2 participants
@cacoco
Member

cacoco commented Sep 28, 2015

No description provided.

@cacoco cacoco added the in progress label Sep 28, 2015

@codecov-io

This comment has been minimized.

codecov-io commented Sep 28, 2015

Current coverage is 93.79%

Merging #258 into master will not affect coverage as of 85e88e2

@@            master    #258   diff @@
======================================
  Files          174     174       
  Stmts         2274    2274       
  Branches       270     270       
  Methods          0       0       
======================================
  Hit           2133    2133       
  Partial          0       0       
  Missed         141     141       

Review entire Coverage Diff as of 85e88e2

Powered by Codecov. Updated on successful CI builds.

cacoco added a commit that referenced this pull request Sep 28, 2015

Merge pull request #258 from twitter/ccoco/AddPoolUtils
finatra/inject - Rename InjectUtils to more specific PoolUtils

@cacoco cacoco merged commit 31798a4 into master Sep 28, 2015

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@cacoco cacoco removed the in progress label Sep 28, 2015

@cacoco cacoco deleted the ccoco/AddPoolUtils branch Sep 28, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment