-
Notifications
You must be signed in to change notification settings - Fork 12
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
MicroProfile Health 3.0 Specification Review #14
Comments
Hola Martin, We forgot to include the final step, the Result Ballot notification after the initiated ballot is sent. Thank YOU! the general template needs to be adjusted as well. :) |
Hola Martin, @xstefank We forgot to include the final step, the Result Ballot notification after the initiated ballot is sent. Thank YOU! the general template needs to be adjusted as well. :) |
PMC approval - https://www.eclipse.org/lists/technology-pmc/msg12125.html. I can't edit the labels. Shouldn't the committers have push rights to this repository? |
+1! I reviewed all the links and they look good! |
+1 Everything seems ok. |
+1 for Jelastic |
@xstefank Same comment I made on the Metrics Issue #15... We need the Compatible Implementation to be "permanent" and reproducible. Just pointing at the Metrics code repo is not sufficient. The idea is to have a binary that can be referenced for a long time in case there are ever any questions about whether this Spec, API, CI, and TCK were properly executed and tested. |
Hi @kwsutter, I've released smallrye-health 3.0.0-RC1 - eclipse/microprofile-health#279. Would it be ok like this? |
Since Kevin is off in the next 4 days, I've checked the build and tck result. I can confirm the build that uses the staged Health 3.0 api was released and all tcks passed. It is ok to proceed with the release. Thank you @xstefank for addressing the issue! |
Thanks @Emily-Jiang. I am going to release MP Health 3.0 now! :) |
Specification issue template
Release Review
The text was updated successfully, but these errors were encountered: