codacy / codacy-analysis-cli-action

GitHub Action for the codacy-analysis-cli
https://github.com/codacy/codacy-analysis-cli
Apache License 2.0
56 stars 18 forks source link

[TS-170] misdetects .sql as TSQL #97

Closed mirabilos closed 10 months ago

mirabilos commented 1 year ago

I get an error because my .sql file is for PostgreSQL and contains PL/pgSQL and your tool misdetects it as Microsoft TSQL and lacks an override

github-actions[bot] commented 1 year ago

Internal ticket created : TS-170

codacy-vrhpires commented 10 months ago

Codacy chooses the tools based strictly on extensions. Currently there are 3 tools for .sql files:

You can choose the specific tools being used with a Codacy configuration file.

mirabilos commented 10 months ago

Vítor Pires dixit:

Codacy chooses the tools based strictly on extensions. Currently there are 3 tools for .sql files:

  • SQLint
  • PMD
  • TSQLint

Huh.

SQLint says it supports PostgreSQL, which would work.

You can choose the specific tools being used with a Codacy configuration file.

How? The linked page doesn’t explain.

Currently, I have:

$ cat .codacy.yml

engines: pylint:

pylintpython3 is used instead

enabled: false

tsqllint:

for Microsoft only

enabled: false

This is seemingly ignored…

The page doesn’t say how to choose the tool for the extension.

bye, //mirabilos -- Infrastrukturexperte • Qvest Digital AG Am Dickobskreuz 10, D-53121 Bonn • https://www.qvest-digital.com/ Telephon +49 228 54881-393 • Fax: +49 228 54881-235 HRB AG Bonn 18196 • USt-ID (VAT): DE274355441 Vorstand: Dr. Stefan Barth, Kai Ebenrett, Boris Esser, Alexander Steeg Vorsitzender Aufsichtsrat: Peter Nöthen