Clojurians Log v2

@shaunlebron I’m looking at #173 - I made two fixes which made that case work, but only at the expense of breaking other test cases around indent/dedent 😞

Actually, only one of my fixes is specific to this problem, the other I think should be made anyway, all tests pass with it.

Ok, I have a fix that passes all current unit tests, and also fixes #173.

But I’m really not sure it’s the right solution, or at least I’m not 100% sure I actually understand the problem.

Unfortunately the markdown test parser can’t handle this case - it requires diff lines to have all the changes adjacent. My JS regexp-fu is probably not up to fixing that.

i.e. it only allows one change per line.

Ok, PR sent for discussion: https://github.com/shaunlebron/parinfer/pull/174

@cfleming: thanks 🎉

will merge this soon, i want to do some manual testing tomorrow and fix the diff annotation parser to allow multiple changes on a line for this

btw, i’ve been working this week on https://github.com/shaunlebron/parinfer/issues/92

i’ve had to rethink a lot of stuff, but it’s looking good so far

Oh nice, that will be a good change.