Page MenuHomePhabricator

Remove backticks from console-formatted strings
AbandonedPublic

Authored by joshuaspence on Nov 24 2015, 8:26 PM.
Tags
None
Referenced Files
F18855427: D14569.id35239.diff
Nov 1 2025, 5:16 AM
F18654672: D14569.diff
Sep 22 2025, 4:34 PM
F18624649: D14569.id.diff
Sep 15 2025, 8:41 PM
Unknown Object (File)
Jul 3 2025, 5:36 PM
Unknown Object (File)
Jun 6 2025, 10:32 AM
Unknown Object (File)
May 27 2025, 9:06 PM
Unknown Object (File)
May 27 2025, 7:55 AM
Unknown Object (File)
May 7 2025, 1:48 AM
Subscribers

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Summary

Remve ` from console-formatted strings because we can't really render them properlyn anyway (most console have a monospaced font by default).

Test Plan

Added unit tests.

Diff Detail

Repository
rPHU libphutil
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 9051
Build 10647: Run Core Tests
Build 10646: arc lint + arc unit

Event Timeline

joshuaspence retitled this revision from to Remove backticks from console-formatted strings.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.

Really what we should do here is ensure that text within backticks is formatted literally. This would solve the issue described in D14563#164583. I'm not really sure how to go about this though.

joshuaspence edited edge metadata.

Seems to work now

I think it's fine/desirable to emit this literal text onto the console:

Run `ls -l` to see files in the current directory.

Specifically, even when not marked up, this is more clear than:

Run ls -l to see files in the current directory.

...since it points out the ends of the token and which parts are and are not components of the command.

I think there's a different fix for D14563#164583, let me look at that.