nfl/react-helmet

Move typespec to this repo

Open

#358 aperta il 25 feb 2018

Vedi su GitHub
 (1 commento) (3 reazioni) (0 assegnatari)JavaScript (706 fork)batch import
enhancementgood first issuehelp wanted

Metriche repository

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

Descrizione

Is there any reason why we can't move the typespec from https://github.com/DefinitelyTyped/DefinitelyTyped to this repo so we do not have to deal with two packages.

We could copy that into this package and add "types": "./lib/main.d.ts" in the package.json

Guida contributor

Direzione di ricerca
Per prima cosa, trova le definizioni di tipo esistenti per react helmet in DefinitelyTyped (https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/react helmet). Copia i file .d.ts pertinenti in questo repository, ad esempio in una directory 'types'. Quindi, aggiorna package.json aggiungendo un campo 'types' che punti al nuovo file di dichiarazione (ad esempio, "./lib/main.d.ts"). Assicurati che i tipi siano allineati con l'API corrente. Controlla i commenti dell'issue per eventuali feedback dei maintainer.
Tech stack
typescriptreact
Dominio
developer experience
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Familiarity with TypeScript type definitionsUnderstanding of npm package.json
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80