Details
- Reviewers
durin42 pulkit - Group Reviewers
hg-reviewers - Commits
- rHG377689cc295f: config: test priority involving alias
Diff Detail
- Repository
- rHG Mercurial
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Event Timeline
tests/test-config.t | ||
---|---|---|
448 | From higher level do you mean the file which is read later or some kind of levels in configs? |
tests/test-config.t | ||
---|---|---|
448 | The file which is read the latest (actually, the value which is read the latest because include make this non-trivial). |
tests/test-config.t | ||
---|---|---|
448 | Can you rephrase the comment then? I confused it with config aliases. |
tests/test-config.t | ||
---|---|---|
448 | I don't know what confuse you, so I am not sure what kind of rewriting we need. |
tests/test-config.t | ||
---|---|---|
448 | I still have no idea what "higher level" means: is the highest level the systemwide config or is that lowest level? Perhaps "most-global" or "repo-local" instead of "higher level". |
This still requires changes to resolve my comments.
(Perhaps baymax could be altered to not refresh patches that are in "needs changes" state, since it clears that bit.)
The series has received multiple comment update that should clarify your request. Can you recheck the series (and clarify your issue if you still have some?)
(Perhaps baymax could be altered to not refresh patches that are in "needs changes" state, since it clears that
In this case, the new version is expected to cover the requested change, so baymax behavior was right there.
I'm still confused here.
tests/test-config.t | ||
---|---|---|
451 | I'm still confused by the "highest level (read later)" here - does this mean the command-templates version is higher, or the ui one is higher? |
tests/test-config.t | ||
---|---|---|
451 | It depends on which one is the last be read. this (should) behave the same as if the file were setting the same value. Except each file set value for different, yet synonyms, config option. (cf initial report of the issue by Yuya a while back). |
From higher level do you mean the file which is read later or some kind of levels in configs?