-
Notifications
You must be signed in to change notification settings - Fork 927
Closed
Labels
good first issueGood for newcomersGood for newcomershelp wantedinfraInternal work not facing public APIsInternal work not facing public APIs
Description
Describe the Feature
Some time ago we started migrating CLI to TS package by package. The infrastructure is already set up, so we can work with TS and Flow files without issues.
Currently all packages except for packages/cli
are converted. We need your help in converting the rest of the codebase 💪.
Work plan
If you'd like to help with migrating any of the packages mentioned below, please leave a comment and we'll assign it to you.
Commands
Commands live in separate directories. It makes sense to tackle them one by one.
- commands/bundle (@Quadriphobs1)
- commands/config (@netochaves)
- commands/doctor (@thib92)
- commands/info (@FLGMwt)
- commands/init (@gvarandas)
- commands/install (@DanielUche)
- commands/link (@anikethsaha)
- commands/server (@Esemesek)
- commands/upgrade (@thymikee)
- commands/index.js (@thymikee)
Tools
Tools are helpers we use throughout the cli
package. They have corresponding test files, which can be found in commands/tools/__tests__
directory. Those should also be converted.
- commands/tools/assertRequiredOptions.js (@konojunya)
- commands/tools/assign.js (@radko93)
- commands/tools/brewInstall.js (@thymikee)
- commands/tools/config/ (@PTROCKI)
- commands/tools/copyAndReplace.js (@luizcieslak)
- commands/tools/copyFiles.js (@sasurau4)
- commands/tools/findSymlinkedModules.js (@pveyes)
- commands/tools/generator/ (@KangYoosam)
- commands/tools/gracefulifyFs.js (@assuncaocharles)
- commands/tools/install.js (@thymikee)
- commands/tools/installPods.js (@thymikee)
- commands/tools/isValidPackageName.js (@radko93)
- commands/tools/loadMetroConfig.js (@radko93)
- commands/tools/loader.js (@thymikee)
- commands/tools/merge.js (@Naturalclar)
- commands/tools/packageManager.js (@moriyuu)
- commands/tools/releaseChecker/ (@FLGMwt)
- commands/tools/replacePathSepForRegex.js (@FLGMwt)
- commands/tools/walk.js (@FLGMwt)
- commands/tools/yarn.js (@FLGMwt)
Root index files in src/
Let's tackle this once we have all commands converted.
- cliEntry.js, index.js, bin.js (@Esemesek)
Esemesek, marvinhagemeister, radko93, konojunya, sasurau4 and 12 moreEsemesek, sasurau4, Naturalclar, felippepuhle, assuncaocharles and 2 moreEsemesek, sasurau4, lucasbento, KangYoosam, felippepuhle and 2 moreEsemesek, sasurau4, KangYoosam, felippepuhle, assuncaocharles and 1 more
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomershelp wantedinfraInternal work not facing public APIsInternal work not facing public APIs