Page MenuHomePhabricator

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

Authored by abosh on Jul 14 2022, 1:14 PM.
Tags
None
Referenced Files
F2195398: D4541.diff
Fri, Jul 5, 10:16 AM
Unknown Object (File)
Sun, Jun 30, 7:58 PM
Unknown Object (File)
Sat, Jun 22, 7:12 AM
Unknown Object (File)
Fri, Jun 21, 9:26 PM
Unknown Object (File)
Fri, Jun 21, 2:52 PM
Unknown Object (File)
Fri, Jun 21, 2:52 PM
Unknown Object (File)
Fri, Jun 21, 2:51 PM
Unknown Object (File)
Fri, Jun 21, 2:45 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
Branch
master
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

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

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.