Implemented `newline-last` rule

rokucommunity / bslint

A linter for BrightScript and BrighterScript.

MIT License

28 stars 14 forks source link

Closed arturocuya closed 2 years ago

arturocuya commented 2 years ago

Also considered naming this eol-last like eslint does, but I think "newline" is more undestandable.

arturocuya commented 2 years ago

@philippe-elsass-deltatre Update:

arturocuya commented 2 years ago

Update:

As Bron suggested, if possible don't assume line ending for fixing. Use the last line ending instead, except when the file is a valid one-liner that needs to be fixed.
Cleaned up code for checking if file is empty.
Simplified how temp files are created in tests.

arturocuya commented 2 years ago

Update: Rebase from master after another PR was merged.