rokucommunity / bslint

A linter for BrightScript and BrighterScript.
MIT License
28 stars 14 forks source link

Logical operator casing #32

Open elsassph opened 3 years ago

elsassph commented 3 years ago

I've seen devs wanting to enforce either lowercase or uppercase for logical operators.

cond = a OR b AND NOT c
' vs
cond = a or b and not c
addison-adler commented 1 week ago

@elsassph Been a few years but this works now!

@TwitchBronBron This one could be marked complete? I'm not sure which release this got added in (since it was opened in 2021) but it does work in present-day 🙂

TwitchBronBron commented 6 days ago

Those formatter settings will definitely change the keywords to the correct case, but I believe the point of this ticket is actually to show a diagnostic when they're wrong. (for running in CI/CD environments where a developer may not have run the formatter before committing).