forked from facebook/astryx
-
Notifications
You must be signed in to change notification settings - Fork 0
Review 5413
Cindy Zhang edited this page Aug 26, 2026
·
1 revision
josephfarina · internal · #5413 · MERGED 2026-08-25
Head 7ecbd8d23d16270dc4a21c6071dd8613c360040e. Seven files, em dashes and curly apostrophes in doc prose.
Approve, no findings. All slots clear.
The thing worth recording is why: the sweep's exemptions are right, and that is the part a mechanical sweep gets wrong. Audited both directions at head — every em dash and curly character still standing is one the body says it deliberately left:
StepContent.doc.mjs:7,8 name / displayName 'Step — Content Slot'
StepperCustomContent.doc.mjs:7,8 name / displayName 'Stepper — Custom Content'
ButtonGroup.doc.mjs:91 CJK '…会被跳过——即 WAI-ARIA APG…'
Step.doc.mjs:84,85 inside a // code comment
themeTargets.doc.mjs:7 the file's own @position docblock header
No code fence, example string, identifier, locale catalog entry or test fixture was touched.
Risk class: low-risk. Fully green — this one predates the pr-visual breakage.