gajus/eslint-plugin-sql

allow to supply custom isSqlQuery helper or use better library

Open

#4 aberto em 1 de set. de 2018

Ver no GitHub
 (6 comments) (4 reactions) (0 assignees)TypeScript (24 forks)github user discovery
help wanted

Métricas do repositório

Stars
 (113 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

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.

Guia do colaborador

Direção de pesquisa
Pesquise as limitações da biblioteca sql parse atual, explore parsers SQL alternativos (por exemplo, node sql parser, sql.js) e projete uma abordagem extensível para o plugin aceitar um helper isSqlQuery personalizado ou uma opção de configuração para usar um parser diferente.
Pilha de tecnologia
typescript
Domain
devtools
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
TypeScriptESLintSQL
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60