gajus/eslint-plugin-sql

allow to supply custom isSqlQuery helper or use better library

Open

#4 ouverte le 1 sept. 2018

Voir sur GitHub
 (6 commentaires) (4 réactions) (0 assignés)TypeScript (24 forks)github user discovery
help wanted

Métriques du dépôt

Stars
 (113 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

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.

Guide contributeur

Direction de recherche
Recherchez les limites de la bibliothèque sql parse actuelle, explorez des analyseurs SQL alternatifs (par exemple, node sql parser, sql.js) et concevez une approche extensible pour que le plugin accepte un helper isSqlQuery personnalisé ou une option de configuration pour utiliser un analyseur différent.
Stack technique
typescript
Domaine
devtools
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
TypeScriptESLintSQL
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60