search

cloudfoundry-attic / cf-abacus

CF usage metering and aggregation
Apache License 2.0
98 stars 86 forks source link

Remove dead code from broker #1128

Closed martin-d-aleksandrov closed 5 years ago

cfdreddbot commented 5 years ago

:white_check_mark: Hey martin-d-aleksandrov! The commit authors and yourself have already signed the CLA.

cf-gitbot commented 5 years ago

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/162238448

The labels on this github issue will be updated when the story is started.

codecov[bot] commented 5 years ago

Codecov Report

Merging #1128 into master will decrease coverage by 0.01%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1128      +/-   ##
==========================================
- Coverage    95.2%   95.19%   -0.02%     
==========================================
  Files         126      126              
  Lines        3486     3474      -12     
==========================================
- Hits         3319     3307      -12     
  Misses        167      167
Impacted Files Coverage Δ
lib/cf/broker/src/auth/admin-token.js 100% <100%> (ø)
lib/cf/broker/src/auth/uaa.js 100% <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 eb602c2...9248ea1. Read the comment docs.