Page MenuHomePhabricator

Remove backticks from console-formatted strings
AbandonedPublic

Authored by joshuaspence on Nov 24 2015, 8:26 PM.
Tags
None
Referenced Files
F14039755: D14569.id35241.diff
Mon, Nov 11, 6:15 AM
F14039754: D14569.id35240.diff
Mon, Nov 11, 6:15 AM
F14039753: D14569.id35239.diff
Mon, Nov 11, 6:15 AM
F14039752: D14569.id.diff
Mon, Nov 11, 6:15 AM
F14039751: D14569.diff
Mon, Nov 11, 6:15 AM
F14035865: D14569.id35241.diff
Sun, Nov 10, 7:50 AM
F14035633: D14569.id35238.diff
Sun, Nov 10, 6:53 AM
F14035612: D14569.id35241.diff
Sun, Nov 10, 6:45 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 9050
Build 10645: Run Core Tests
Build 10644: 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.