-
Notifications
You must be signed in to change notification settings - Fork 4.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move expo-doctor over from expo-cli #25503
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two minor comments, but looking good overal!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is just a suggestion we could do, see #25424 (comment) for more info.
Co-authored-by: Cedric van Putten <me@bycedric.com>
Co-authored-by: Cedric van Putten <me@bycedric.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two small nits, but looking good! Thanks for adding
Co-authored-by: Cedric van Putten <me@bycedric.com>
Co-authored-by: Cedric van Putten <me@bycedric.com>
# Why Add expo-doctor to this repository for future maintenance # How - Copied over expo-doctor folder from expo-cli/packages - Adapted changes from expo#25422 to switch scripts to `expo-module` equivalents - updated unit tests to properly mock `spawnAsync` - Fixed linter warnings # Questions - I'm not sure what, if anything needs to be added to CI jobs, since Doctor only has unit tests currently - Kind of guessing as to what **tsconfig.json** should look like, mostly copied @expo/cli with a few added package-specific flags # Test Plan - [x] Test locally - [ ] see if CI passes --------- Co-authored-by: Cedric van Putten <me@bycedric.com>
Why
Add expo-doctor to this repository for future maintenance
How
@expo/babel-preset-cli
withexpo-module-scripts
#25422 to switch scripts toexpo-module
equivalentsspawnAsync
Questions
Test Plan