This is an archive of the discontinued Mercurial Phabricator instance.

tweak-default: no longer enable the experimental return code
ClosedPublic

Authored by marmoute on Feb 10 2021, 2:01 PM.

Details

Summary

The change is quite new and undocumented (since it is experimental) so it seems
premature to make it available in tweak default. In addition, I am not sure the
new return code are frozen yet (eg: some of the initial feedback have not been
incorporated). Before the release I doubled check (probably with Martin) that
they were not enabled by default and got replied that they were only enabled in
the tests. Have I been aware that they have been also enabled in tweak default I
would I have lobbied to delay that.

I discovered they were in tweak default from users feedback. They found it an
unpleasant and unexpected surprise of 5.7.

So I suggest we no enable enable this experimental feature for Mercurial 5.7 and
revisit this later, when the feature will be more mature.

Diff Detail

Repository
rHG Mercurial
Branch
stable
Lint
No Linters Available
Unit
No Unit Test Coverage

Event Timeline

marmoute created this revision.Feb 10 2021, 2:01 PM

⚠ This is intended for stable ⚠

martinvonz accepted this revision.Feb 10 2021, 2:58 PM
martinvonz added a subscriber: martinvonz.

I agree. My commit message in 21733e8c doesn't mention changing tweakdefaults, so I wonder if I added that by mistake. I remember also being surprised myself that we had started rolling out the change internally before I thought we had (we have tweakdefaults enabled). Sorry that I didn't think of it then that maybe it shouldn't have been in there.

This revision is now accepted and ready to land.Feb 10 2021, 2:58 PM