search

sasjs / lint

Linting and formatting for SAS® code
https://sasjs.io
MIT License
25 stars 2 forks source link

`maxLineLengthCards` feature #47

Closed allanbowe closed 1 year ago

allanbowe commented 3 years ago

Datalines cannot be split over multiple lines, as each line represents an input data record. So it makes no sense to include them in the line length limit.

image

There are multiple 'flavours' of datalines. Each will begin / end with a line that begins with the markers below

section begin section end
datalines; ;
datalines4; ;;;;
cards; ;
cards4; ;;;;
parmcards; ;
parmcards4; ;;;;
sasutils commented 2 years ago

You probably do want to have a check for long data lines. First there are limits to what SAS will accept as a line length for a program file Second if your datalines are longer than your program lines then they belong in a separate text file.

allanbowe commented 2 years ago

Hmm, you do have a point about the batch line length limit.

allanbowe commented 1 year ago

Ok so the plan will be to have a maxLineLengthCards (to work only with datalines) that will default to max(maxLineLengthCards,maxLineLength)

sasjsbot commented 1 year ago

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

The release is available on:

Your semantic-release bot :package::rocket:

sabhas commented 1 year ago

via Hubstaff User: Sabir Hassan

Project: SASjs lint - https://app.hubstaff.com/projects/2310878 Date Range: 01/10/23 - 01/11/23 Work session total: 5:16:01 Billable: Yes

Grand total: 5:16:01