1
95%
main: 95%

Ran 30 Jul 2024 11:51AM UTC

Files 17

Run time 0s

Badge

Embed ▾

Committed 30 Jul 2024 11:46AM UTC coverage: 95.185%. Remained the same

Job # 10161784058.1

Build Type

push

github

Committed by

jjatria

Commit Message

Use bigfloat instead of bignum in ratio sampler

This is a minor change to sidestep an issue caused at a distance
by loading this pragma. The effect can be seen in the following
snippets:

    $ perl -Mbignum -E '
        use Math::BigFloat;
        say Math::BigFloat->new(42)->ffround;
    '
    Can't locate object method "ffround" via package "Math::BigInt" at -e line 1.

    $ perl -Mbigfloat -E '
        use Math::BigFloat;
        say Math::BigFloat->new(42)->ffround;
    '
    42

The issue (reported in [GH #16]) means that loading
OpenTelemetry::SDK (and indirectly loading `bignum`) would
at-a-distance break any code that was attempting to (directly or
indirectly) call `ffround` on a Math::BigFloat object.

Although this is most definitely not an OpenTelemetry::SDK bug,
it's unclear whose bug it _is_, and unclear how easy it would be
to fix upstream, so this solution which avoids the problem is
probably the best short-term approach.

Since `bifloat` was only added in Math::BigInt 0.65, we are
reporting this as a new dependency, to ensure it is available.

[GH #16]: https://github.com/jjatria/perl-opentelemetry-sdk/issues/16

Run Details

257 of 270 relevant lines covered (95.19%)

4.95 hits per line

jjatria / perl-opentelemetry-sdk / 10161784058 / 1
95%
main: 95%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 10161784058.1

jjatria / perl-opentelemetry-sdk / 10161784058 / 1 95% main: 95%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 10161784058.1

jjatria / perl-opentelemetry-sdk / 10161784058 / 1
95%
main: 95%

README BADGES
x