Page MenuHomePhabricator

[chore] Replace `'` with `’` in user-displayed text
ClosedPublic

Authored by abosh on Jul 14 2022, 1:14 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 12, 1:16 AM
Unknown Object (File)
Tue, Nov 12, 1:16 AM
Unknown Object (File)
Tue, Nov 12, 1:16 AM
Unknown Object (File)
Tue, Nov 12, 1:16 AM
Unknown Object (File)
Mon, Nov 11, 2:51 AM
Unknown Object (File)
Thu, Nov 7, 10:19 PM
Unknown Object (File)
Sat, Nov 2, 1:15 PM
Unknown Object (File)
Sat, Nov 2, 1:15 PM

Details

Summary

Replaced single quote with apostrophe in user-displayed text. Wasn't sure the best way to title this, but basically this diff only replaces the single quotes in text that can get displayed in a modal, the app, or an error message that isn't an invariant.

Test Plan

Used this regex to search for all instances of ' surrounded by "":

image.png (324×524 px, 25 KB)

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

abosh edited the summary of this revision. (Show Details)
abosh added inline comments.
lib/facts/genesis.json
1 ↗(On Diff #14503)

Not sure where this or testers.json is used anymore, but it's human-readable/user-displayed text somewhere, so I changed it.

Seems reasonable. It might be hard to keep it this way in the codebase since the ' key is a single quote in code editors.

There's probably some way to enforce the rule, but I don't think it really matters enough to go through that effort.

This revision is now accepted and ready to land.Jul 14 2022, 5:49 PM

Yeah, it just depends on how stringent future code reviewers want to be.