Skip to content

docs: remove all emoji, filler words, and first-person across versioned docs and i18n#323

Closed
mesutoezdil wants to merge 2 commits intoProject-HAMi:masterfrom
mesutoezdil:fix/remaining-emoji-filler-i18n
Closed

docs: remove all emoji, filler words, and first-person across versioned docs and i18n#323
mesutoezdil wants to merge 2 commits intoProject-HAMi:masterfrom
mesutoezdil:fix/remaining-emoji-filler-i18n

Conversation

@mesutoezdil
Copy link
Copy Markdown
Contributor

@mesutoezdil mesutoezdil commented May 7, 2026

Removes all remaining style violations from versioned_docs (v1.3.0-v2.8.0) and i18n/zh.

Emoji removed:

  • 💖 from ladder.md and adopters.md (all versions, English and Chinese)
  • ⚠️ from volcano-vgpu warning blocks and upgrade.md (Chinese i18n)
  • 👉 from Chinese kcd-beijing-2026 blog post (9 occurrences)
  • ✅ / ❌ from device-supported.md tables replaced with Yes / No (all versions)

Filler phrases removed:

  • 'Note that', 'Please note that', 'In summary,' across developer and userguide docs
  • "It's worth noting that" from faq.md (v1.3.0, v2.4.1, v2.5.0)
  • "Let's" from verify-hami.md and contribute-docs.md

First-person replaced:

  • contributor/: contributing.md, contribute-docs.md, governance.md, cherry-picks.md, ladder.md, adopters.md
  • developers/: dynamic-mig.md, scheduling.md, protocol.md
  • userguide/: enflame, iluvatar, cambricon, metax, configure, prerequisites, get-started

163 files across versioned_docs and i18n/zh. Supersedes PR #322.

…ed docs and i18n

Apply style cleanup to versioned_docs (v1.3.0-v2.8.0) and i18n/zh:

- Remove emoji: 💖 from ladder/adopters, ⚠️ from volcano-vgpu and upgrade docs,
  👉 from Chinese kcd-beijing blog, ✅/❌ from device-supported tables (replaced with Yes/No)
- Remove filler phrases: 'Note that', 'Please note', 'In summary', 'It's worth noting', "Let's"
- Replace first-person 'we/our' with third-person in contributor, developer, and userguide docs
- 165 files across versioned_docs and i18n/zh

Signed-off-by: mesutoezdil <mesudozdil@gmail.com>
@hami-robot hami-robot Bot requested review from wawa0210 and windsonsea May 7, 2026 19:16
@hami-robot
Copy link
Copy Markdown
Contributor

hami-robot Bot commented May 7, 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 wawa0210 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

@netlify
Copy link
Copy Markdown

netlify Bot commented May 7, 2026

Deploy Preview for project-hami ready!

Name Link
🔨 Latest commit 5aa908f
🔍 Latest deploy log https://app.netlify.com/projects/project-hami/deploys/69fce6ee4c8b9a000860f811
😎 Deploy Preview https://deploy-preview-323--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.

Replace ***text*** with **text** across versioned_docs and i18n/zh
userguide files. Fix "the HAMI scheduler" to "the HAMi scheduler" in
protocol.md across v1.3.0, v2.4.1, v2.5.0, and v2.5.1.

Signed-off-by: mesutoezdil <mesudozdil@gmail.com>
@mesutoezdil
Copy link
Copy Markdown
Contributor Author

Superseded by #326 which consolidates all style fixes into a single conflict-free PR.

@mesutoezdil mesutoezdil closed this May 7, 2026
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.

1 participant