sindresorhus/electron-store

Types error with setting deserialize in v6.0.0

Open

#133 aperta il 29 lug 2020

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (150 fork)batch import
bughelp wanted

Metriche repository

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

Descrizione

Hi there, we override deserialize to use yaml instead; this previously use to work but a change in the typings of conf now means it doesn't compile in typescript.

serialize: yaml.safeDump,
deserialize: yaml.safeLoad,

Error:

Type '(str: string, opts?: LoadOptions | undefined) => string | object | undefined' is not assignable to type 'Deserialize<Record<string, any>>'.
  Type 'string | object | undefined' is not assignable to type 'Record<string, any>'.
    Type 'undefined' is not assignable to type 'Record<string, any>'.ts(2322)

Guida contributor

Direzione di ricerca
Il problema segnala un errore di tipo TypeScript quando si assegnano funzioni di deserializzazione/serializzazione personalizzate utilizzando yaml in electron store v6.0.0. L'errore indica che il tipo di ritorno di yaml.safeLoad (string | object | undefined) non è assegnabile al tipo Deserialize<Record<string, any>> previsto. La correzione probabilmente comporta l'aggiornamento della definizione di tipo nella dipendenza conf (utilizzata da electron store) per consentire undefined nel tipo di ritorno. I file rilevanti sono le definizioni di tipo per conf e possibilmente index.d.ts in electron store. Un manutentore dovrebbe rivedere il tipo Deserialize e decidere se ampliarlo o suggerire una soluzione alternativa.
Tech stack
typescriptnodejselectron
Dominio
desktop
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.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
TypeScriptGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60