EmptyLines cop does auto-correction.

[dblock]

No description provided.

[coveralls]

Coverage decreased (-0.1%) when pulling a0de8239192680d1bb20e4c4789b1406ad68c3ff on dblock:autocorrect-empty-lines into 167b49c on bbatsov:master.

[jonas054]

I think the problem is that you're looping over offences in autocorrect. The method will be called once per offence. Take a look at TrailingBlankLines where lines are removed in autocorrect.

[dblock]

@jonas054 right on, thanks! My code caused clobbering (remove on the same range was called twice) which caused the detection to try to highlight the issue, which caused it to raise an exception when being rendered, because it cannot render something that spans more than one line.

Now fixed, this is good to merge.

[coveralls]

Coverage decreased (-0.1%) when pulling 66b61c4 on dblock:autocorrect-empty-lines into 62a7059 on bbatsov:master.

[coveralls]

Coverage decreased (-0.1%) when pulling 66b61c4 on dblock:autocorrect-empty-lines into 62a7059 on bbatsov:master.

[bbatsov]

👍