Skip to content

Commit 8047912

Browse files
CongLuanTranUzaaft
andauthored
feat(sql)!: Dont force ansi dialect (#1628)
* fix(sql): Remove forced ansi dialect for sqlfluff * fix(sql): Modify README to add instruction for `sqlfluff` * Update README.md --------- Co-authored-by: Uzair Aftab <48220549+Uzaaft@users.noreply.github.com>
1 parent 668273d commit 8047912

File tree

2 files changed

+3
-7
lines changed

2 files changed

+3
-7
lines changed

lua/astrocommunity/pack/sql/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,4 +7,4 @@ This plugin pack does the following:
77
- `go` must be in your `PATH` and executable.
88
- formatting is disabled due to https://github.com/sqls-server/sqls/issues/149
99
- Adds [sqls.nvim](https://github.com/nanotee/sqls.nvim) for language specific tooling
10-
- Adds `sqlfluff` for both formatting and linting (using the ANSI dialect)
10+
- Adds [sqlfluff](https://docs.sqlfluff.com) for both formatting and linting

lua/astrocommunity/pack/sql/init.lua

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -34,12 +34,8 @@ return {
3434

3535
opts.handlers.sqlfluff = function()
3636
local null_ls = require "null-ls"
37-
null_ls.register(null_ls.builtins.diagnostics.sqlfluff.with {
38-
extra_args = { "--dialect", "ansi" },
39-
})
40-
null_ls.register(null_ls.builtins.formatting.sqlfluff.with {
41-
extra_args = { "--dialect", "ansi" },
42-
})
37+
null_ls.register(null_ls.builtins.diagnostics.sqlfluff)
38+
null_ls.register(null_ls.builtins.formatting.sqlfluff)
4339
end
4440
end,
4541
},

0 commit comments

Comments
 (0)