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.

Sign up for GitHub

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

Jump to bottom

Prefer `Metrics.metric` rather than modify the current class. #6

Merged

ioquatix merged 1 commit into main from non-invasive-metrics

Jun 19, 2023

Member

ioquatix commented Jun 19, 2023 •

edited

Loading

I started to feel a little concerned about monkey patching user code, so I decided to change this interface.

Types of Changes

Maintenance.

Contribution

I added tests for my changes.
I tested my changes locally.
I agree to the Developer's Certificate of Origin 1.1.


          Prefer Metrics.metric rather than modify the current class.

9b48ded

ioquatix merged commit aa83725 into main

ioquatix deleted the non-invasive-metrics branch

June 19, 2023 03:55

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

Labels

None yet