-
Notifications
You must be signed in to change notification settings - Fork 44
Open
Labels
bugSomething isn't workingSomething isn't working
Description
What happened?
LT05
is marked fixable:
Lines 1418 to 1422 in 18965ac
**Code:** `LT05` | |
**Groups:** `all`, `core`, `layout` | |
**Fixable:** Yes |
However this simple SQL:
SELECT COUNT(*)
FROM policies
WHERE
is_active_policy IS TRUE AND active_policy = 'Authorized' AND exclude_from_checks IS TRUE;
Results in LT05
and no change to the line length.
❯ sqruff fix sqruff_issue.sql
== [sqruff_issue.sql] FAIL
L: 4 | P: 5 | LT05 | Line is too long (94 > 80).
| [layout.long_lines]
The linter processed 1 file(s).
All Finished 📜 🎉
If I make a simple change to the SQL:
SELECT COUNT(*)
FROM policies
WHERE
is_active_policy IS TRUE AND active_policy = 'Authorized'
AND exclude_from_checks IS TRUE;
Then all is well.
❯ sqruff fix sqruff_issue.sql
1 files processed, nothing to fix.
Is there a reason sqruff
can't do the same? For context I never used the sqlfluff
project that this appears to be based on so maybe I'm misunderstanding how this should work.
Version
❯ sqruff --version
sqruff 0.29.3
Checked in Playground
- I have checked this bug in the playground.
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't working