Skip to content

docs(cli): recast em dashes in the internationalization guide#4076

Merged
josephfarina merged 1 commit into
mainfrom
navi/docs/slop-17
Jul 20, 2026
Merged

docs(cli): recast em dashes in the internationalization guide#4076
josephfarina merged 1 commit into
mainfrom
navi/docs/slop-17

Conversation

@josephfarina

Copy link
Copy Markdown
Contributor

What

The internationalization reference guide (packages/cli/docs/internationalization.doc.mjs, added in #4016) used em dashes in 11 prose strings. This recasts each with the punctuation the sentence calls for:

  • colon for a definition or list intro
  • semicolon for two independent clauses
  • comma for an appositive
  • parentheses for a nested aside

Prose strings only. No technical claim was reworded and no information was removed. Code examples, the JSX/JS comments inside them, and the backtick pseudo-locale glyph ⟦...⟧ are left as-is (they are code, not prose).

Verification

  • pnpm --filter @astryxdesign/core typecheck:docs passes
  • pnpm exec tsc --project packages/cli/tsconfig.template-docs.json --noEmit passes
  • astryx docs internationalization renders cleanly; the only remaining em dash/ellipsis characters in the output are inside code examples and the pseudo-locale glyph

Night Watch — Doc Reviewer

The internationalization reference guide (packages/cli/docs/internationalization.doc.mjs)
used em dashes in 11 prose strings. Recast each with the punctuation the
sentence calls for: colon for a definition or list intro, semicolon for two
independent clauses, comma for an appositive, and parentheses for a nested
aside. Prose only; no technical claims changed. Code examples, JSX comments,
and the backtick pseudo-locale glyph are untouched.

Found during Night Watch doc review.
@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Meta Open Source bot. label Jul 19, 2026
@vercel

vercel Bot commented Jul 19, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
astryx Ready Ready Preview, Comment Jul 19, 2026 9:39am

Request Review

@github-actions

Copy link
Copy Markdown
Contributor

PR Analysis Report

📚 Storybook Preview

View Storybook for this PR
GitHub Pages may take up to a minute to hydrate after deploy.

🧪 Sandbox Preview

View Sandbox for this PR
GitHub Pages may take up to a minute to hydrate after deploy.

No new or modified components detected.

Bundle Size Summary

Package Size (ESM) Size (CJS) Gzipped
@astryxdesign/core N/A 4.7KB 0B

Accessibility Audit

Status: No accessibility violations detected.


Generated by PR Enrichment workflow | Storybook | Sandbox | View full report

@josephfarina
josephfarina merged commit 6c88144 into main Jul 20, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Meta Open Source bot.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant