A recent change has included *.md
in the line length checks. I'm now getting warnings that my line is exceeding 80 chars, causing tests to fail. Unfortunately, the line in question is markdown (a build status image with a link) and so cannot be broken down into multiple lines, or the markdown wont build correctly. The failing line is:
[![Build Status](https://travis-ci.org/FreelyGive/decoupled_auth.png)](https://travis-ci.org/FreelyGive/decoupled_auth)
I have worked round it for now by using references, but even that is still on the brink of breaking (especially if I were to use a title as well as alt text). I think the ideal would be to do some kind of exclude for lines that are start with markdown that needs to stay on a single line. Possible examples are (pardon my iffy regex):
- Headers:
^#.+
- Links:
^\[.+\](\[|\().+(\]|\))$
([Title](Url)
and[Title][ref]
) - Images:
^\!\[.+\](\[|\().+(\]|\))$
(![Alt](Url)
and[Alt][ref]
) - References (used in images and links):
^\[.+\]: .+
- Tables(?):
^|.+|$
Comments
Comment #2
bircherI was about to open the same issue :D
So another problem is that in markdown you can't even disable the code sniffer with something like:
Comment #3
andrewbelcher CreditAttribution: andrewbelcher at FreelyGive commented@bircher the workaround for you which I think should pass tests is:
Comment #7
klausiPushed a fix, thanks for reporting!
Comment #8
andrewbelcher CreditAttribution: andrewbelcher at FreelyGive commentedAwesome, thanks!