cjoudrey/graphql-schema-linter

Add basic validation for inline configuration inputs

Open

#238 aperta il 4 lug 2020

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (62 fork)batch import
bugenhancementhelp wanted

Metriche repository

Star
 (669 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

When specifying an invalid rule in an inline configuration, graphql-schema-linter should let the user know.

For example, this should raise an error:

# lint-disable some-invalid-rule

type Query {
  field: String
}

Perhaps the process could terminate with an error like:

The inline configuration on line 1 of `filename.graphql` references a non-existing rule: some-invalid-rule

Similarly, we should gracefully handle when an inline configuration is specified without any rules.

This currently crashes:

# lint-disable

type Query {
  field: String
}

We could either (1) make it raise an error like above or (2) support this as a feature and make it disable all rules.

/cc @mshwery in case you're bored 😄

Guida contributor

Direzione di ricerca
Esamina la logica di parsing dei commenti inline e convalida i nomi delle regole rispetto all'elenco delle regole disponibili.
Tech stack
javascript
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptGraphQL
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60