dense-analysis / dense-analysis/ale

[Improvement Suggestion][sqlfluff] Allow other config file other than `.sqlfluff`

Open
#4,554 3 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Vim Script
Stars
14k
Forks
1.5k
Avg merge
17h 49m
Merged PRs (30d)
1

Description

`sqlfluff` looks for the [list of these files](https://docs.sqlfluff.com/en/stable/configuration.html#configuration-files) here automatically and should not require a hardcoded .sqlfluff to be looked up in the [linter](https://github.com/dense-analysis/ale/blob/29e5ffe35a2fa1ca5ba19eebfe7cdd0191c16e39/ale_linters/sql/sqlfluff.vim#L22) and in the [fixer](https://github.com/dense-analysis/ale/blob/29e5ffe35a2fa1ca5ba19eebfe7cdd0191c16e39/autoload/ale/fixers/sqlfluff.vim#L13). Users of other config file and in my case pyproject.toml have no choice but to migrate to `.sqlfluff`

I want to suggest that we remove the constraint on allowing only `.sqlfluff` file as config file and let `sqlfluff` search for it.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.