Add Flow types to website source
Open
Nobody has claimed this yet.
difficulty: medium
good first issue
in-progress
type: website
- Dominant language
- JavaScript
- Stars
- 11.8k
- Forks
- 7.9k
- Avg merge
- 1d 11h
- Merged PRs (30d)
- 11
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.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing the existing work in commit 1de4c66 and the components under www. Run yarn flow to see which website source files are checked and where PropTypes remain. Done means the components use Flow types and yarn flow checks them successfully to prevent regressions.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript, react
- Domain
- documentation, frontend
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100