Commit b55c0cea authored by Anton Shestakov's avatar Anton Shestakov
Browse files

obshistory: omit duplicate notes of obsolescence operations

When folding changesets, it's possible to provide a --note flag to e.g. fold or
prune, which will store a note for that operation. But since there would be
more than one marker for the fold operation and the provided note would be
stored in all of them, _markersnotes() previously would return the same text
message many times. We don't really want to see the same thing over and over,
so let's make notes a set().

This will also make unrelated operations with the same note show the message
only once, which is probably an improvement too.
parent 3847dedd12d7
Pipeline #5699 passed with stage
in 5 minutes and 49 seconds
......@@ -711,7 +711,7 @@
def _markersnotes(markers):
markersmeta = [dict(m[3]) for m in markers]
notes = [meta.get(b'note') for meta in markersmeta]
notes = {meta.get(b'note') for meta in markersmeta}
return sorted(note for note in notes if note)
EFFECTMAPPING = util.sortdict([
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment