docs: strip decorative emoji across the repo - #16
Merged
Merged
Conversation
387 emoji were spread across 20 files, including comment banners in source and pictographs printed into the user's build log. A library should not put "PROCESSING FILE" behind a magnifying glass in somebody's CI output. Removed 148 decorative ones. What stayed encodes state rather than decorating it: - checkmark, cross and warning sign in docs and in publish.sh CLI output - red/yellow/green priority markers in TESTING_ROADMAP.md - the byline heart in readme.md Source files were stripped completely, comments and console output alike. Box drawing (tree diagrams in CONTRIBUTING.md) and arrows are not emoji and were left alone. An earlier pass reflowed whitespace as well and mangled JSDoc continuation lines, breaking 18 tests; this one only touches the emoji character and one adjacent space. Also refreshes the TESTING_ROADMAP header, which still claimed "113/113 tests passing" and "Planning Phase". It is now 1268 tests, with fuzz and mutation coverage that postdates the document.
canadianeagle
force-pushed
the
docs/emoji-sweep
branch
from
July 27, 2026 21:46
0025d60 to
e3f6162
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follows #14, which only covered readme.md. This does the rest of the repo.
387 emoji across 20 files, including comment banners in source and pictographs printed straight into the user's build log. A published plugin should not put
PROCESSING FILEbehind a magnifying glass in somebody's CI output.Removed 148. What stayed encodes state rather than decorating it:
publish.shCLI outputTESTING_ROADMAP.mdreadme.mdSource files stripped completely, comments and console output alike.
Box drawing (the tree diagrams in
CONTRIBUTING.md) and arrows are not emoji and were left alone.A note on how this was done
My first pass also reflowed whitespace, which treated JSDoc continuation lines (
*) as markdown bullets and mangled them, breaking 18 tests. Reverted and redone touching only the emoji character and one adjacent space. Verified clean.Also
TESTING_ROADMAP.mdstill said "113/113 tests passing" and "Status: Planning Phase". It is now 1268 tests with fuzz and mutation coverage that postdates the document, so the header is updated and marked partly superseded.Verification
lint PASS,
tsc --noEmitPASS, 1268/1268 tests, build PASS, CJS entry imports.