Mercurial > evolve
changeset 4939:7aba58dc4b73
obslog: use plural name "effects" for list of all effects
Same reasoning as the previous patch, but here the singular was used
for the list instead.
This patch also renames the variable that represents the list of
effects to plural "effects" to reduce confusion.
author | Martin von Zweigbergk <martinvonz@google.com> |
---|---|
date | Thu, 07 Nov 2019 15:00:57 -0800 |
parents | d8033f75a52e |
children | 93c89814a5d8 |
files | hgext3rd/evolve/obshistory.py tests/test-evolve-cycles.t tests/test-evolve-obshistory-amend-then-fold.t tests/test-evolve-obshistory-amend.t tests/test-evolve-obshistory-content-divergent.t tests/test-evolve-obshistory-fold.t tests/test-evolve-obshistory-lots-of-splits.t tests/test-evolve-obshistory-phase-divergent.t tests/test-evolve-obshistory-split.t |
diffstat | 9 files changed, 44 insertions(+), 44 deletions(-) [+] |
line wrap: on
line diff
--- a/hgext3rd/evolve/obshistory.py Thu Nov 07 13:21:20 2019 -0800 +++ b/hgext3rd/evolve/obshistory.py Thu Nov 07 15:00:57 2019 -0800 @@ -521,27 +521,27 @@ except ValueError: effectflag = None if effectflag: - effect = [] + effects = [] # XXX should be a dict if effectflag & DESCCHANGED: - effect.append(b'description') + effects.append(b'description') if effectflag & METACHANGED: - effect.append(b'meta') + effects.append(b'meta') if effectflag & USERCHANGED: - effect.append(b'user') + effects.append(b'user') if effectflag & DATECHANGED: - effect.append(b'date') + effects.append(b'date') if effectflag & BRANCHCHANGED: - effect.append(b'branch') + effects.append(b'branch') if effectflag & PARENTCHANGED: - effect.append(b'parent') + effects.append(b'parent') if effectflag & DIFFCHANGED: - effect.append(b'content') + effects.append(b'content') - if effect: - fmteffect = fm.formatlist(effect, b'effect', sep=b', ') - fm.write(b'effect', b'(%s)', fmteffect) + if effects: + fmteffect = fm.formatlist(effects, b'effect', sep=b', ') + fm.write(b'effects', b'(%s)', fmteffect) if len(succnodes) > 0: fm.plain(b' as ') @@ -649,27 +649,27 @@ effectflag |= int(ef) if effectflag: - effect = [] + effects = [] # XXX should be a dict if effectflag & DESCCHANGED: - effect.append(b'description') + effects.append(b'description') if effectflag & METACHANGED: - effect.append(b'meta') + effects.append(b'meta') if effectflag & USERCHANGED: - effect.append(b'user') + effects.append(b'user') if effectflag & DATECHANGED: - effect.append(b'date') + effects.append(b'date') if effectflag & BRANCHCHANGED: - effect.append(b'branch') + effects.append(b'branch') if effectflag & PARENTCHANGED: - effect.append(b'parent') + effects.append(b'parent') if effectflag & DIFFCHANGED: - effect.append(b'content') + effects.append(b'content') - if effect: - fmteffect = fm.formatlist(effect, b'effect', sep=b', ') - fm.write(b'effect', b'(%s)', fmteffect) + if effects: + fmteffect = fm.formatlist(effects, b'effect', sep=b', ') + fm.write(b'effects', b'(%s)', fmteffect) if len(succnodes) > 0: fm.plain(b' as ')
--- a/tests/test-evolve-cycles.t Thu Nov 07 13:21:20 2019 -0800 +++ b/tests/test-evolve-cycles.t Thu Nov 07 15:00:57 2019 -0800 @@ -300,7 +300,7 @@ *, (glob) 0 ], - "effect": [ + "effects": [ "description", "parent", "content" @@ -323,7 +323,7 @@ *, (glob) 0 ], - "effect": [ + "effects": [ "description", "parent", "content" @@ -346,7 +346,7 @@ *, (glob) 0 ], - "effect": [ + "effects": [ "description", "parent", "content" @@ -369,7 +369,7 @@ *, (glob) 0 ], - "effect": [ + "effects": [ "description", "parent", "content" @@ -393,7 +393,7 @@ *, (glob) 0 ], - "effect": [ + "effects": [ "description", "parent", "content" @@ -416,7 +416,7 @@ *, (glob) 0 ], - "effect": [ + "effects": [ "description", "parent", "content"
--- a/tests/test-evolve-obshistory-amend-then-fold.t Thu Nov 07 13:21:20 2019 -0800 +++ b/tests/test-evolve-obshistory-amend-then-fold.t Thu Nov 07 15:00:57 2019 -0800 @@ -162,7 +162,7 @@ *, (glob) 0 (glob) ], - "effect": [ + "effects": [ *, (glob) *, (glob) "content" @@ -185,7 +185,7 @@ *, (glob) 0 (glob) ], - "effect": [ + "effects": [ "description" ], "operation": "amend", @@ -206,7 +206,7 @@ *, (glob) 0 (glob) ], - "effect": [ + "effects": [ "description", "content" ],
--- a/tests/test-evolve-obshistory-amend.t Thu Nov 07 13:21:20 2019 -0800 +++ b/tests/test-evolve-obshistory-amend.t Thu Nov 07 15:00:57 2019 -0800 @@ -137,7 +137,7 @@ *, (glob) 0 (glob) ], - "effect": [ + "effects": [ "description", "content" ], @@ -182,7 +182,7 @@ *, (glob) 0 (glob) ], - "effect": [ + "effects": [ *, (glob) "content" ],
--- a/tests/test-evolve-obshistory-content-divergent.t Thu Nov 07 13:21:20 2019 -0800 +++ b/tests/test-evolve-obshistory-content-divergent.t Thu Nov 07 15:00:57 2019 -0800 @@ -129,7 +129,7 @@ *, (glob) 0 (glob) ], - "effect": [ + "effects": [ "description" ], "operation": "amend", @@ -144,7 +144,7 @@ *, (glob) 0 (glob) ], - "effect": [ + "effects": [ "description" ], "operation": "amend", @@ -293,7 +293,7 @@ *, (glob) 0 (glob) ], - "effect": [ + "effects": [ "description" ], "operation": "amend", @@ -308,7 +308,7 @@ *, (glob) 0 (glob) ], - "effect": [ + "effects": [ "description" ], "operation": "amend",
--- a/tests/test-evolve-obshistory-fold.t Thu Nov 07 13:21:20 2019 -0800 +++ b/tests/test-evolve-obshistory-fold.t Thu Nov 07 15:00:57 2019 -0800 @@ -182,7 +182,7 @@ *, (glob) 0 (glob) ], - "effect": [ + "effects": [ "description", "content" ], @@ -204,7 +204,7 @@ *, (glob) 0 (glob) ], - "effect": [ + "effects": [ "description", "parent", "content"
--- a/tests/test-evolve-obshistory-lots-of-splits.t Thu Nov 07 13:21:20 2019 -0800 +++ b/tests/test-evolve-obshistory-lots-of-splits.t Thu Nov 07 15:00:57 2019 -0800 @@ -201,7 +201,7 @@ *, (glob) 0 (glob) ], - "effect": [ + "effects": [ "parent", "content" ], @@ -241,7 +241,7 @@ *, (glob) 0 (glob) ], - "effect": [ + "effects": [ "parent", "content" ],
--- a/tests/test-evolve-obshistory-phase-divergent.t Thu Nov 07 13:21:20 2019 -0800 +++ b/tests/test-evolve-obshistory-phase-divergent.t Thu Nov 07 15:00:57 2019 -0800 @@ -102,7 +102,7 @@ *, (glob) 0 (glob) ], - "effect": [ + "effects": [ "description" ], "operation": "amend", @@ -205,7 +205,7 @@ 0, 0 ], - "effect": [ + "effects": [ "description" ], "operation": "amend",