Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SQL Support #128

Closed
thehesiod opened this issue Jun 18, 2020 · 4 comments · Fixed by #528
Closed

SQL Support #128

thehesiod opened this issue Jun 18, 2020 · 4 comments · Fixed by #528
Labels
enhancement New feature or request

Comments

@thehesiod
Copy link

we have a lot of SQL in our codebase and would be nice to lint these as well.

not sure which linter would be the best

@thehesiod thehesiod added the enhancement New feature or request label Jun 18, 2020
@LifeIsStrange
Copy link

In https://github.com/github/super-linter/issues/115
I mentioned https://github.com/diffplug/spotless
which would allow to solve simultaneously Java, Kotlin, Scala, Groovy and SQL support in a unified way.
For SQL it would use https://github.com/diffplug/spotless/blob/main/lib/src/main/java/com/diffplug/spotless/sql/DBeaverSQLFormatterStep.java
so from this tool
https://github.com/dbeaver/dbeaver/wiki/SQL-Formatting

@jars
Copy link

jars commented Jun 18, 2020

Also check out: https://github.com/alanmcruickshank/sqlfluff

@github-actions
Copy link
Contributor

This issue has been automatically marked as stale because it has not had recent activity.
It will be closed in 14 days if no further activity occurs.
Thank you for your contributions.

If you think this issue should stay open, please remove the O: stale 🤖 label or comment on the issue.

@github-actions github-actions bot added the O: stale 🤖 Stale issue/pr label Jul 19, 2020
@lowlydba
Copy link

lowlydba commented Jul 19, 2020

Bump to prevent stale issue tagging.

@github-actions github-actions bot removed the O: stale 🤖 Stale issue/pr label Jul 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants