gajus/eslint-plugin-sql

allow to supply custom isSqlQuery helper or use better library

Open

#4 aperta il 1 set 2018

Vedi su GitHub
 (6 commenti) (4 reazioni) (0 assegnatari)TypeScript (24 fork)github user discovery
help wanted

Metriche repository

Star
 (113 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Toying with the plugin, I'm coming across valid SQL queries that aren't recognized, such as:

SELECT * FROM article a WHERE NOT a.archived

-- or

SELECT * FROM article a WHERE (a.authz & 8)::bool

One solution would be to either allow users to supply their own SQL parser (though I haven't checked if eslint allows to pass functions as settings values in .eslintrc.js), or replace the sql-parse, which seems unmaintained, with something else.

Guida contributor

Direzione di ricerca
Ricerca le limitazioni dell'attuale libreria sql parse, esplora parser SQL alternativi (es. node sql parser, sql.js) e progetta un approccio estensibile per consentire al plugin di accettare un helper isSqlQuery personalizzato o un'opzione di configurazione per utilizzare un parser diverso.
Tech stack
typescript
Dominio
devtools
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
TypeScriptESLintSQL
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60