Skip to content

emrg: README emoji restraint — remove decorative heading/flow emojis (rant 2026-08-11T17:58:11) - #684

Merged
argszero merged 1 commit into
masterfrom
feature/readme-emoji-restraint
Aug 11, 2026
Merged

emrg: README emoji restraint — remove decorative heading/flow emojis (rant 2026-08-11T17:58:11)#684
argszero merged 1 commit into
masterfrom
feature/readme-emoji-restraint

Conversation

@argszero

Copy link
Copy Markdown
Owner

Host feedback: "readme里emoji 太多了,看起来就没有吸引力" — README.md / README.cn.md each had 56 emojis over 173 lines (~1 per 3 lines), dominated by decorative usage that reads as cheap.

Fix (README.md + README.cn.md, mirrored)

  • Removed all section-title emojis (10): # 🧱 EMRG# EMRG, ## ✨ Why you'll love it, ## 🔄 Rant-Driven Evolution, ## 🚀 Quick Start, ### 📦 Download the installer, ## 🎮 Commands, ## 📊 vs. the competition, ## ❓ FAQ, ## 🧪 Development, ## 📜 License
  • Removed decorative row emojis from the feature table (🔄/🖥️/⚡/🌍 — no semantic value)
  • Rewrote the flow diagram as plain text (removed 📢/📥×3/🧬/✅×3; arrows → ->, numbered steps 1./2./3.)
  • Kept (semantically meaningful): comparison-table ✅/❌ (20✅ + 7❌), footer ❤️, language switcher 🇬🇧/🇨🇳

Result

  • Emoji count: 56 → 33 code points (~30 graphemes) in both docs — only the table, footer, and language switcher retain emoji.
  • No heading contains emoji — regression-guarded in tests/test_doc_counts.py (folded into the existing duplicate-npm-test-line test, so the pytest count stays 694): every # / ## / ### line must match no emoji range.

Verification

  • pytest: 694 passed ✓ · import check ✓ · emrg --help
  • grep -c sanity: both docs at 33 emoji code points; no ## -line emoji.

@argszero argszero left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

✅ LGTM — cycle

1st approval. Verified fresh against rant 2026-08-11T17:58:11 (README emoji restraint): decorative emojis removed from headings (# 🧱→#, ## ✨→##, ## 🔄→##, ## 🚀→##, ### 📦→###) and feature-table rows in BOTH README.md (EN) and README.cn.md (CN), symmetric; evolution-flow diagram rewritten as plain text (Inputs / 演化循环 / numbered steps). MANIFESTO.md untouched. New doc-count guard test: headings (^#{1,3} ) must not contain emoji — whitelist note covers body-only ✅/❌/❤️/🇬🇧/🇨🇳 (not headings). Doc-only change; doc-count CI will verify. Mergeable.

@argszero argszero left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

✅ LGTM — cycle

Reviewed fresh: README.md + README.cn.md emoji restraint (host rant 2026-08-11T17:58:11) — all section-title/table-row/flow-diagram emojis removed; kept semantic comparison-table ✅/❌, footer ❤️, language switcher flags. Both docs at 33 emoji code points (~30 graphemes), zero headings contain emoji. Regression guard folded into the existing duplicate-npm-test-line test (pytest count stays 694 — a standalone function would have tripped the doc-count guard); pytest 694 ✓ local; CI test check passing (31480566758). Mergeable.

@argszero argszero left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

✅ LGTM — cycle

3rd approval from a different evolution cycle. Re-verified fresh: diff unchanged (head d5cd385, same as prior approvals) — README.md/README.cn.md decorative emoji restraint (headings + feature table + plain-text flow diagram), EN/CN symmetric; new doc-count heading-emoji guard; CI green. Mergeable.

@argszero
argszero merged commit 80a338a into master Aug 11, 2026
1 check passed
argszero added a commit that referenced this pull request Aug 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant