facebookarchive/prepack

Make all files flow strict...

Open

#1941 aperta il 11 mag 2018

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (520 fork)batch import
help wantedrefactoring

Metriche repository

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

Descrizione

... one by one.

Flow recently got a "strict" mode. We should make all our files "flow strict". This is the master issue to track progress until all files are flow strict.

We introduced flow strict to Prepack in #1692, and made the first file flow strict: https://github.com/facebook/prepack/pull/1692/files#diff-5c8b199a98cb05fcfa67ad0a331c165b

So, for each files, this starts by changing

* @flow */

to

* @flow strict */

and then fixing up the type errors (if any) that will emerge when running

yarn flow

One practical problem is that we have a giant cycle, and strict files can only import other strict files.

Guida contributor

Direzione di ricerca
Seleziona un file che non è ancora strict flow, cambia la sua intestazione da @flow a @flow strict, quindi esegui `yarn flow` per correggere eventuali errori di tipo. Assicurati che tutti i moduli importati siano anche strict flow.
Tech stack
javascript
Dominio
tooling
Tipo issue
Refactoring
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
FlowJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60