bcherny/json-schema-to-typescript

Cannot pass JSONSchema7 schema to compile as it requires JSONSchema4

Open

#359 aperta il 23 dic 2020

Vedi su GitHub
 (13 commenti) (9 reazioni) (0 assegnatari)TypeScript (449 fork)github user discovery
enhancementhelp wantedin discussion

Metriche repository

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

Descrizione

Currently the types are using JSONSchema4 as the type for schema, which means TypeScript won't let you pass objects of type JSONSchema7 due to type differences.

Specifically, exclusiveMaximum & exclusiveMinimum are typed as boolean | undefined in JSONSchema4, but as number | undefined in JSONSchema7. (there probably are more differences - I've not compared the two further than that).

I think the right type to use would be a union of the different schemas supported, i.e JSONSchema7 | JSONSchema4.

Guida contributor

Direzione di ricerca
Trova la definizione del tipo per il parametro 'schema' nella funzione compile e cambialo da JSONSchema4 a un'unione di JSONSchema4 | JSONSchema7, quindi aggiorna eventuali logiche dipendenti dal tipo se necessario.
Tech stack
typescript
Dominio
tooling
Tipo issue
Bug
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
TypeScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70