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

Grammar and style review for the TLS reference guide #45821

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

MichalMaler
Copy link
Contributor

Providing a brief review after some new content was added to this guide.

@MichalMaler MichalMaler requested a review from jmartisk January 23, 2025 11:41
@quarkus-bot quarkus-bot bot added area/docstyle issues related for manual docstyle review area/documentation labels Jan 23, 2025
@MichalMaler MichalMaler force-pushed the tls-registry-patch-review branch from de36ebf to 803abce Compare January 23, 2025 11:44
Copy link

github-actions bot commented Jan 23, 2025

🎊 PR Preview c91718e has been successfully built and deployed to https://quarkus-pr-main-45821-preview.surge.sh/version/main/guides/

  • Images of blog posts older than 3 months are not available.
  • Newsletters older than 3 months are not available.

This comment has been minimized.

@MichalMaler MichalMaler force-pushed the tls-registry-patch-review branch from 803abce to f48b734 Compare January 23, 2025 13:34
@MichalMaler
Copy link
Contributor Author

Thank you, @cescoffier and @jmartisk, for your review and for moving this forward! Epic co-work, as always. Cheers!

This comment has been minimized.

@MichalMaler MichalMaler added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Jan 23, 2025
Signed-off-by: Michal Maléř <[email protected]>
@MichalMaler MichalMaler force-pushed the tls-registry-patch-review branch from f48b734 to 99197ff Compare January 23, 2025 15:51
Copy link

quarkus-bot bot commented Jan 23, 2025

Status for workflow Quarkus Documentation CI

This is the status report for running Quarkus Documentation CI on commit 99197ff.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/docstyle issues related for manual docstyle review area/documentation triage/waiting-for-ci Ready to merge when CI successfully finishes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants