[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