Remove all Flow library definitions from flow-typed/
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 12.4k
- Forks
- 2.4k
- PR merge metrics
- No merged PRs in 30d
Description
https://github.com/mapbox/mapbox-gl-js/pull/7031 moves type definitions that are local to this project out of `flow-typed` and into `src`, so that they must be imported explicitly. Following up on that, we should remove the remaining libdefs as well as a step towards #6963. (Necessary because in a downstream project, Flow would not look for libdefs in our flow-typed/ directory.)
- [ ] mapbox-gl-supported.js https://github.com/mapbox/mapbox-gl-supported/pull/21
- [ ] mapbox-unitbezier.js https://github.com/mapbox/unitbezier/pull/2
- [ ] pbf.js https://github.com/mapbox/pbf/pull/91
- [ ] point-geometry.js
- [ ] shelf-pack.js
- [ ] unitbezier.js
- [ ] vector-tile.js
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 inspecting the remaining definitions in flow-typed/: mapbox-gl-supported.js, mapbox-unitbezier.js, pbf.js, point-geometry.js, shelf-pack.js, unitbezier.js, and vector-tile.js. Review the referenced upstream pull requests and issue #6963, then remove the remaining libdefs once their definitions are available in the intended locations; done means no remaining library definitions stay under flow-typed/.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- tooling
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100