JS Style Guide: line length exception for comments with long urls or long regex? about contribute.jquery.org

[gibson042]

Yes, I'd like to see such an exception for regex literals and URLs, although the former may eventually be rescinded by minification mapping new RegExp( "literal" + "literal" ) to an actual regex literal.

[scottgonzalez]

I'm ok with this.

[mikesherov]

I'll open a PR for this shortly. In the meantime, I'm going to move forward with this: jscs-dev/node-jscs#418

[scottgonzalez]

Sounds good. I can't wait to finally land JSCS in all projects.

[markelog]

mmm, why for the comments?

[mikesherov]

@markelog this is preventative for comments that contain long URLS. It seems based on this discussion that the comment exception was OK. If I misread this discussion, please correct me.

[scottgonzalez]

Yeah, the comment exception is ok. I was originally going to suggest using a URL shortener, but I think allowing long URLs in comments is better than relying on a third party URL shortener to be around forever.

[markelog]

So comments shouldn't be more than 100 chars with exception for URLs?

[scottgonzalez]

Correct.

[mikesherov]

Oh, I misunderstood. I put in a generic exception for comment length. I suppose I need to make the rule a bit smarter to only allow the exception if the comment contains a url.

[mikesherov]

@markelog I'll update that a bit later today.