-
Notifications
You must be signed in to change notification settings - Fork 4
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
Chapter 7: MicroProfile Metrics #16
Conversation
Adding content for chapter07
Awaiting review feedback and approval. |
@donbourne @Channyboy can you review |
Incorporating Review feedbacks.
Addressing review comments
Resolving comments about Counter
updating as per review feedback.
updating as per review comments.
resolving review comments
resolving review comments
@donbourne can you double check the new changes to see whether you can approve this PR? thanks |
Double checked from myside. @donbourne @Emily-Jiang - can you please re-review and approve this now. |
@ttelang , I will have a look, but may not be until later this week. |
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.
A few minor things to fix.
Fixes for following review comments: - https://github.com/eclipse/microprofile-tutorial/pull/16/files#r1774157042 - https://github.com/eclipse/microprofile-tutorial/pull/16/files#r1774155140 - https://github.com/eclipse/microprofile-tutorial/pull/16/files#r1774153130 - https://github.com/eclipse/microprofile-tutorial/pull/16/files#r1774147832 - https://github.com/eclipse/microprofile-tutorial/pull/16/files#r1774147151
Fixed typo
Adding content for chapter07