[PATCH 0 of 3 v2] diff: add collapse to simple diff and make it configurable
Jan Heylen
heyleke at gmail.com
Thu Dec 24 20:32:33 UTC 2015
Hi,
this is v2 patchset that makes simple diffs collapsable and it's default behaviour configurable.
Apparently the collapsing code was broken since a while, I reverted the change that broke it (just this part).
As said by Mads, I messed up testing of v1 of this patch, I did rebaseing and refactoring of the patch in a different source tree,
but forgot to switch the virtualenv, so I ended up testing the same un-refactored and unrebased tree all the time (without noticing :-s).
That is also the reason I didn't saw that upstream the collapsing code was broken since a while...
Sorry for the confusion, please review V2 of the patch, and as said before:
the part where the default behavior is hardcoded in pull requests:
I don't think this is pullable, but maybe if this is made a "setting".
So, the main purpose of putting this last part on the mailinglist is a RFC for the "settings" part.
Should and how should I make this configurable?
But please, do already take in the first 2 patches, if OK.
regards,
Jan
More information about the kallithea-general
mailing list