This is an archive of the discontinued Mercurial Phabricator instance.

status: add a config knob for setting default of --terse
ClosedPublic

Authored by durin42 on May 20 2018, 1:57 AM.

Details

Summary

I want --terse=u basically 100% of the time, but there's not a good
way to do that before this patch.

I'm very unhappy with how the default value for --terse looks rigt
now, but it does *work*. The alternative would be to define an
"optional string" flag type using fancyopts.customopt and then use
that, leaving the default as None. Does anyone have a strong
preference for that, or a better idea?

Diff Detail

Repository
rHG Mercurial
Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

durin42 created this revision.May 20 2018, 1:57 AM
pulkit accepted this revision.May 20 2018, 4:06 PM
pulkit added a subscriber: pulkit.

I'm very unhappy with how the default value for --terse looks rigt
now, but it does *work*. The alternative would be to define an
"optional string" flag type using fancyopts.customopt and then use
that, leaving the default as None. Does anyone have a strong
preference for that, or a better idea?

I am fine with the current approach. Queued the series, many thanks!

This revision was automatically updated to reflect the committed changes.
In D3627#57185, @pulkit wrote:

I'm very unhappy with how the default value for --terse looks rigt
now, but it does *work*. The alternative would be to define an
"optional string" flag type using fancyopts.customopt and then use
that, leaving the default as None. Does anyone have a strong
preference for that, or a better idea?

I am fine with the current approach. Queued the series, many thanks!

Note that the current value is visible in hg help -v status:

-t --terse VALUE         show the terse output (EXPERIMENTAL) (default:
                         nothing)

Probably not a big deal, but I thought I'd mention it.

mercurial/help/config.txt
446

Typo in "condenes"