3
100%
master: 100%

Ran 17 Jun 2019 05:12PM UTC

Files 32

Run time 1s

Badge

Embed ▾

Committed 17 Jun 2019 05:01PM UTC coverage: 100.0%. First build

Job # 2.5.3

Build Type

push

travis-ci

Committed by

dmagliola

Commit Message

Remove `concurrent-ruby` dependency

We added this dependency to be able to use `ReadWriteLock`, which is
a reentract lock. We did not need the Read/Write locking distinction,
and at the time, we didn't know that `Monitor` was reentrant, so this
seemed to be the best solution.

Knowing that `Monitor` is reentrant, we can get rid of the dependency,
and simply use `Monitor` instead.

Note that we keep `concurrent-ruby` as a *dev* dependency, because the
performance benchmarks use other primitives provided by it to synvhronize
their threads, but it's not needed for specs to pass, or for production use.

Signed-off-by: Daniel Magliola <danielmagliola@gocardless.com>

Run Details

1417 of 1417 relevant lines covered (100.0%)

14.98 hits per line

prometheus / client_ruby / 421 / 3
100%
master: 100%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 421.3 (2.5.3)

prometheus / client_ruby / 421 / 3 100% master: 100%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 421.3 (2.5.3)

prometheus / client_ruby / 421 / 3
100%
master: 100%

README BADGES
x