lint icon indicating copy to clipboard operation
lint copied to clipboard

ignore max line length in comments

Open allanbowe opened this issue 4 years ago • 2 comments

Comments can contain long urls, also table markdown, that cannot be shortened

We should provide an automatic exception for these in the line length limit.

image

allanbowe avatar May 05 '21 14:05 allanbowe

Might be better to have the ability to specify a separate limit rather than totally ignore long lines in comment blocks.

Or perhaps something that can detect when it is a single long "word" that is exceeding the line limit. That would handle the example of the long URL in the comment block.

sasutils avatar Mar 10 '22 18:03 sasutils

@sasutils - I like your idea of ignoring a "word" (ie, string without a space)! Thanks for the suggestion!

allanbowe avatar Mar 10 '22 18:03 allanbowe

On retrospect, the single word idea won't work as we also need it to work for table markdown (where each row must be on one line, and can contain spaces)

So your first idea is on the table. A new config option maxLineLengthHeader (to work only with the header section) that will default to max(maxLineLengthHeader,maxLineLength)

allanbowe avatar Jan 05 '23 16:01 allanbowe

via Hubstaff User: Sabir Hassan

Project: SASjs lint - https://app.hubstaff.com/projects/2310878 Date Range: 01/09/23 - 01/10/23 Work session total: 9:19:59 Billable: Yes

Grand total: 9:19:59

sabhas avatar Jan 11 '23 14:01 sabhas

:tada: This issue has been resolved in version 2.2.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

sasjsbot avatar Jan 12 '23 16:01 sasjsbot