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.
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
Bump micrometer from 1.6.2 to 1.8.4 #132
Conversation
@yeikel on the vertx user group you said you had issues with metrics when upgrading to this version but it seems our test suite does not detect any regression. Can you elaborate on the issues? |
Are you referring to I am not the one who reported problems with this version. The email you're referring to was signed by "Clive" |
@yeikel yes I was. Sorry for the confusion. |
No problem! Thank you for checking Let me know if you publish any snapshot for this upgrade so I can also test it as well |
I'm waiting for a reply from the user. Besides, this would be a Vert.x
4.3.0 upgrade so I can't merge yet (master branches still target 4.2).
Anyway, given this is a simple POM version, you could test right away by
changing the micrometer dependencies in your project's POM file.
… Message ID: ***@***.***
com>
|
Please note that |
70ea9c7
to
d2b7516
Compare
Note that |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @yeikel
Motivation:
Maintenance/ Keep dependencies up to date