Skip to content

fix: convert WARNING, NOTICE and Important blockquotes to Docusaurus admonitions#386

Open
mesutoezdil wants to merge 1 commit into
Project-HAMi:masterfrom
mesutoezdil:fix/remaining-blockquote-admonitions
Open

fix: convert WARNING, NOTICE and Important blockquotes to Docusaurus admonitions#386
mesutoezdil wants to merge 1 commit into
Project-HAMi:masterfrom
mesutoezdil:fix/remaining-blockquote-admonitions

Conversation

@mesutoezdil
Copy link
Copy Markdown
Contributor

Converts 48 remaining informal blockquotes to Docusaurus admonitions across docs/, versioned_docs/, and i18n/. WARNING blocks become :::warning, NOTICE blocks become :::note, and Important blocks become :::caution. No content changes.

@hami-robot
Copy link
Copy Markdown
Contributor

hami-robot Bot commented May 14, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: mesutoezdil
Once this PR has been reviewed and has the lgtm label, please assign windsonsea for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@hami-robot hami-robot Bot requested review from archlitchi and rootsongjc May 14, 2026 12:16
@hami-robot hami-robot Bot added the size/L label May 14, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented May 14, 2026

Deploy Preview for project-hami ready!

Name Link
🔨 Latest commit 88922e3
🔍 Latest deploy log https://app.netlify.com/projects/project-hami/deploys/6a06cb8a2e63c000083e154f
😎 Deploy Preview https://deploy-preview-386--project-hami.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown
Member

@windsonsea windsonsea left a comment

Choose a reason for hiding this comment

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

Those indentations are required.

Comment thread docs/installation/webui-installation.md Outdated
Comment on lines +45 to +49
:::caution

Replace `externalPrometheus.address` with the in-cluster Prometheus URL that your environment uses.

:::
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
:::caution
Replace `externalPrometheus.address` with the in-cluster Prometheus URL that your environment uses.
:::
:::caution
Replace `externalPrometheus.address` with the in-cluster Prometheus URL that your environment uses.
:::

Comment on lines +45 to +49
:::caution

Replace `externalPrometheus.address` with the in-cluster Prometheus URL that your environment uses.

:::
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
:::caution
Replace `externalPrometheus.address` with the in-cluster Prometheus URL that your environment uses.
:::
:::caution
Replace `externalPrometheus.address` with the in-cluster Prometheus URL that your environment uses.
:::

Comment on lines +45 to +49
:::caution

Replace `externalPrometheus.address` with the in-cluster Prometheus URL that your environment uses.

:::
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
:::caution
Replace `externalPrometheus.address` with the in-cluster Prometheus URL that your environment uses.
:::
:::caution
Replace `externalPrometheus.address` with the in-cluster Prometheus URL that your environment uses.
:::

Comment on lines +45 to +49
:::caution

Replace `externalPrometheus.address` with the in-cluster Prometheus URL that your environment uses.

:::
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
:::caution
Replace `externalPrometheus.address` with the in-cluster Prometheus URL that your environment uses.
:::
:::caution
Replace `externalPrometheus.address` with the in-cluster Prometheus URL that your environment uses.
:::

Comment on lines +45 to +49
:::caution

Replace `externalPrometheus.address` with the in-cluster Prometheus URL that your environment uses.

:::
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
:::caution
Replace `externalPrometheus.address` with the in-cluster Prometheus URL that your environment uses.
:::
:::caution
Replace `externalPrometheus.address` with the in-cluster Prometheus URL that your environment uses.
:::

@mesutoezdil mesutoezdil force-pushed the fix/remaining-blockquote-admonitions branch from 17a277e to b19bf06 Compare May 15, 2026 07:20
…admonitions

Converts 48 informal blockquote-style markers to Docusaurus admonitions
across docs/, versioned_docs/, and i18n/:

- WARNING blocks become :::warning
- NOTICE blocks become :::note
- Important blocks become :::caution

Multi-line blockquotes are merged into a single admonition block.
Indented admonition content is indented to match the opener level.
Trailing newlines added where missing. No content changes.

Signed-off-by: mesutoezdil <mesudozdil@gmail.com>
@mesutoezdil mesutoezdil force-pushed the fix/remaining-blockquote-admonitions branch from 6f76adb to 88922e3 Compare May 15, 2026 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants