[PR #3228] [MERGED] fix(docs): render README source notices as GitHub alerts #5656

Closed
opened 2026-10-31 01:08:20 +00:00 by bsakaguchi · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/morluto/rea/pull/3228
Author: @N0zoM1z0
Created: 10/14/2026
Status: ✅ Merged
Merged: 10/14/2026
Merged by: @N0zoM1z0

Base: main ← Head: fix/readme-important-alert-20261014


📝 Commits (1)

  • ce86853 fix(docs): render README source notices as GitHub alerts

📊 Changes

20 files changed (+80 additions, -0 deletions)

View changed files

📝 README.md (+4 -0)
📝 docs/readme/README_ar.md (+4 -0)
📝 docs/readme/README_de.md (+4 -0)
📝 docs/readme/README_el.md (+4 -0)
📝 docs/readme/README_es.md (+4 -0)
📝 docs/readme/README_fa.md (+4 -0)
📝 docs/readme/README_fr.md (+4 -0)
📝 docs/readme/README_id.md (+4 -0)
📝 docs/readme/README_it.md (+4 -0)
📝 docs/readme/README_ja.md (+4 -0)
📝 docs/readme/README_ko.md (+4 -0)
📝 docs/readme/README_pl.md (+4 -0)
📝 docs/readme/README_pt-BR.md (+4 -0)
📝 docs/readme/README_ru.md (+4 -0)
📝 docs/readme/README_th.md (+4 -0)
📝 docs/readme/README_tr.md (+4 -0)
📝 docs/readme/README_uk.md (+4 -0)
📝 docs/readme/README_vi.md (+4 -0)
📝 docs/readme/README_zh-TW.md (+4 -0)
📝 docs/readme/README_zh.md (+4 -0)

📄 Description

Summary

The official-source IMPORTANT notice appeared as literal [!IMPORTANT] text because it was inside the centered HTML header. GitHub alerts cannot be nested in other elements (official syntax).

Close the header before the notice and reopen it afterward. Apply the same markup fix to all 20 README editions. The notice stays in place; its text, links, other centered content and Persian RTL attributes are preserved. No translated wording or website files are changed.

Verification

  • GitHub's actual Markdown API reproduces the original failure: zero IMPORTANT alert elements, literal [!IMPORTANT] displayed.
  • Before/after rendering of all 20 complete READMEs: each corrected file has exactly one markdown-alert-important element and no raw marker. Rendered link/image targets are identical.
  • Removing only the two inserted HTML boundaries recovers each original source exactly.
  • Scoped pinned oxfmt --check and git diff --check passed.

No new tests were added or local repository test suites run. No runtime, schema, provider, configuration or release-version change. CI still needs to validate this head.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/morluto/rea/pull/3228 **Author:** [@N0zoM1z0](https://github.com/N0zoM1z0) **Created:** 10/14/2026 **Status:** ✅ Merged **Merged:** 10/14/2026 **Merged by:** [@N0zoM1z0](https://github.com/N0zoM1z0) **Base:** `main` ← **Head:** `fix/readme-important-alert-20261014` --- ### 📝 Commits (1) - [`ce86853`](https://github.com/morluto/rea/commit/ce86853afd25bb644ed200047fb3e2438c984521) fix(docs): render README source notices as GitHub alerts ### 📊 Changes **20 files changed** (+80 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+4 -0) 📝 `docs/readme/README_ar.md` (+4 -0) 📝 `docs/readme/README_de.md` (+4 -0) 📝 `docs/readme/README_el.md` (+4 -0) 📝 `docs/readme/README_es.md` (+4 -0) 📝 `docs/readme/README_fa.md` (+4 -0) 📝 `docs/readme/README_fr.md` (+4 -0) 📝 `docs/readme/README_id.md` (+4 -0) 📝 `docs/readme/README_it.md` (+4 -0) 📝 `docs/readme/README_ja.md` (+4 -0) 📝 `docs/readme/README_ko.md` (+4 -0) 📝 `docs/readme/README_pl.md` (+4 -0) 📝 `docs/readme/README_pt-BR.md` (+4 -0) 📝 `docs/readme/README_ru.md` (+4 -0) 📝 `docs/readme/README_th.md` (+4 -0) 📝 `docs/readme/README_tr.md` (+4 -0) 📝 `docs/readme/README_uk.md` (+4 -0) 📝 `docs/readme/README_vi.md` (+4 -0) 📝 `docs/readme/README_zh-TW.md` (+4 -0) 📝 `docs/readme/README_zh.md` (+4 -0) </details> ### 📄 Description ## Summary The official-source IMPORTANT notice appeared as literal `[!IMPORTANT]` text because it was inside the centered HTML header. GitHub alerts cannot be nested in other elements ([official syntax](https://docs.github.com/en/get-started/writing-on-github/getting-started-with-writing-and-formatting-on-github/basic-writing-and-formatting-syntax#alerts)). Close the header before the notice and reopen it afterward. Apply the same markup fix to all 20 README editions. The notice stays in place; its text, links, other centered content and Persian RTL attributes are preserved. No translated wording or website files are changed. ## Verification - GitHub's actual Markdown API reproduces the original failure: zero IMPORTANT alert elements, literal `[!IMPORTANT]` displayed. - Before/after rendering of all 20 complete READMEs: each corrected file has exactly one `markdown-alert-important` element and no raw marker. Rendered link/image targets are identical. - Removing only the two inserted HTML boundaries recovers each original source exactly. - Scoped pinned `oxfmt --check` and `git diff --check` passed. No new tests were added or local repository test suites run. No runtime, schema, provider, configuration or release-version change. CI still needs to validate this head. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
bsakaguchi 2026-10-31 01:08:20 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
morluto/rea#5656
No description provided.