gajus/eslint-plugin-sql

allow to supply custom isSqlQuery helper or use better library

Open

#4 geöffnet am 1. Sept. 2018

Auf GitHub ansehen
 (6 Kommentare) (4 Reaktionen) (0 zugewiesene Personen)TypeScript (24 Forks)github user discovery
help wanted

Repository-Metriken

Stars
 (113 Stars)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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.

Contributor Guide

Research-Richtung
Recherchieren Sie die Einschränkungen der aktuellen sql parse Bibliothek, erkunden Sie alternative SQL Parser (z. B. node sql parser, sql.js) und entwerfen Sie einen erweiterbaren Ansatz, damit das Plugin einen benutzerdefinierten isSqlQuery Helper oder eine Konfigurationsoption zur Verwendung eines anderen Parsers akzeptieren kann.
Tech Stack
typescript
Domain
devtools
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
TypeScriptESLintSQL
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60