Skip to content
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

Winduprule 476 spring metrics quarkus #429

Merged

Conversation

jonathanvila
Copy link
Member

@jonathanvila jonathanvila commented Apr 1, 2020

@mrizzi
Copy link
Member

mrizzi commented Apr 1, 2020

retest

@PhilipCattanach
Copy link
Contributor

@jonathanvila - The rules worked fine. Nice to see the new Micrometer (embedded) Technology tag on the Application List. The issues are nicely worded and the links work fine and are appropriate. I was a little surprised that the test data was jar file rather than just source files. The tests took a long time to run due to the decompilation effort. Also I was wondering about whether these rules would lend themselves to quickfixes? Thanks.

jonathan added 3 commits April 23, 2020 19:56
removed quickfixes
added extra rule for Timed annotation
changed effort for annotations replacement
added not-has-hint to global search
changed Javadoc links to the latest version of the artifact
Copy link
Member

@mrizzi mrizzi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

commented for fixing presentation issues.

Copy link
Member

@mrizzi mrizzi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM @jonathanvila thanks

@mrizzi mrizzi merged commit 3cb85e2 into windup:master Apr 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants