zostay / raku-Prometheus-Client

A Prometheus Client for Raku
1 stars 2 forks source link

Label test #3

Closed kbucheli closed 4 years ago

zostay commented 4 years ago

Okay, even though my brain is mush I figured out the test failures for this and #4. The issue is that you have changed some classes to does Base does Descriptor, but since Base already does Descriptor that causes some versions of Raku to choke on the second role application. If you are going to make the change to use Base instead of Collector then you should also drop the second does Descriptor at the same time.

kbucheli commented 4 years ago

Sorry, that is a copy of PR #4 from another of my branches...