This is an archive of the discontinued Mercurial Phabricator instance.

scm-prompt: style cleanups
ClosedPublic

Authored by ryanmce on Sep 22 2017, 10:28 AM.
Tags
None
Subscribers

Details

Reviewers
stash
Group Reviewers
Restricted Project
Commits
rFBHGXbfe2ae441934: scm-prompt: style cleanups
Test Plan

rt

Diff Detail

Repository
rFBHGX Facebook Mercurial Extensions
Branch
default
Lint
Lint OK
Unit
Unit Tests OK

Event Timeline

ryanmce created this revision.Sep 22 2017, 10:28 AM
stash accepted this revision.Sep 22 2017, 10:35 AM
stash added a subscriber: stash.
stash added inline comments.
scripts/scm-prompt.sh
85–88

Is it just style too, or it has some practical implications?

This revision is now accepted and ready to land.Sep 22 2017, 10:35 AM
ryanmce added inline comments.
scripts/scm-prompt.sh
85–88

If there was a space it would have failed before. I do a more thorough cleanup later in the stack, and I should have saved this til then, but the re-ordering conflicts sucked so I didn't.

This revision was automatically updated to reflect the committed changes.