reactjs/react.dev

Add Flow types to website source

Open

#24 ouverte le 6 oct. 2017

Voir sur GitHub
 (16 commentaires) (0 réactions) (0 assignés)JavaScript (7 898 forks)github user discovery
difficulty: mediumgood first issuein-progresstype: website

Métriques du dépôt

Stars
 (11 763 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

Replace PropTypes with Flow types for our components in www (and make sure yarn flow checks these types to protect against regressions).

I got the ball rolling with commit 1de4c66

This Twitter thread also has some interesting discussion for auto-generating Flow types for Gatsby's graphql queries.

Guide contributeur

Direction de recherche
Examinez le commit initial 1de4c66 pour comprendre le modèle de conversion, puis remplacez PropTypes par des types Flow pour tous les composants du répertoire www. Exécutez yarn flow pour vous assurer que la vérification de type réussit.
Stack technique
javascript
Domaine
frontend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitJavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70