This is an archive of the discontinued Mercurial Phabricator instance.

config: use level to properly deal with value priority
ClosedPublic

Authored by marmoute on Jan 29 2021, 8:40 PM.

Details

Summary

A higher priority alias will now take precedence over lower priority ones.

This was a requirements step before using alias more widely, especially to
rename existing and established config option.

Diff Detail

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

Event Timeline

marmoute created this revision.Jan 29 2021, 8:40 PM
Alphare added a subscriber: Alphare.Feb 8 2021, 5:20 AM
Alphare added inline comments.
tests/test-config.t
476–477

Should this comment be dropped now?

marmoute added inline comments.Feb 9 2021, 12:45 PM
tests/test-config.t
476–477

good catch, it should. (dropped in latest version).

Alphare accepted this revision.Feb 9 2021, 3:37 PM
baymax updated this revision to Diff 25519.Feb 9 2021, 6:29 PM

✅ refresh by Heptapod after a successful CI run (🐙 💚)

baymax updated this revision to Diff 25576.Feb 11 2021, 1:46 PM

✅ refresh by Heptapod after a successful CI run (🐙 💚)

pulkit accepted this revision.Mar 4 2021, 6:27 AM
This revision is now accepted and ready to land.Mar 4 2021, 6:27 AM
This revision was automatically updated to reflect the committed changes.