Page MenuHomePhabricator

Remove backticks from console-formatted strings
AbandonedPublic

Authored by joshuaspence on Nov 24 2015, 8:26 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Sep 14, 3:12 PM
Unknown Object (File)
Sat, Sep 7, 3:22 AM
Unknown Object (File)
Wed, Sep 4, 11:25 AM
Unknown Object (File)
Sun, Sep 1, 12:47 PM
Unknown Object (File)
Sat, Aug 31, 4:49 AM
Unknown Object (File)
Fri, Aug 30, 1:36 PM
Unknown Object (File)
Fri, Aug 30, 6:31 AM
Unknown Object (File)
Thu, Aug 29, 4:13 PM
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 9048
Build 10641: Run Core Tests
Build 10640: 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.