Should we alleviate the 72 characters rule for lines with long URLs and note about it in the COLLABORATOR_GUIDE.md?
Example: https://github.com/nodejs/node/pull/17107
Refs: https://github.com/nodejs/core-validate-commit/issues/24
I think there aren't really any workaround for this anyway? This is similar to the ignoreUrls option of max-length in eslint.
@joyeecheung We can use some URL shortener, but this can mess things up.
Should we alleviate the 72 characters rule for lines with long URLs and note about it in the COLLABORATOR_GUIDE.md?
I think this is what we've always done anyway, so why not make it explicit.
+1 to what @gibfahn said.
Most helpful comment
I think this is what we've always done anyway, so why not make it explicit.