FunctionBodyLengthRule should not count comment lines

[proth]

lines that start with a "//" after removing all leading whitespaces should not be treated as line inside a function. Otherwise the obvious way to fix violations of that rule would be to write less comments.

[ghost]

+1.

It should also ignore headerdoc documentation comments.

[jpsim]

I like this idea.

lines that start with a "//" after removing all leading whitespaces should not be treated as line inside a function. Otherwise the obvious way to fix violations of that rule would be to write less comments.

SwiftLint can actually detect all kinds of comments (//, /* */, ///, /** */), so really we could just not count lines that only have comment-like tokens.

[kevindelord]

+1

Looks even better with all kind of comments: //, /* */, ///, /** */

[scottrhoyt]

+1 here. I've already adapted some of my commenting style to avoid this.