- 
                Notifications
    You must be signed in to change notification settings 
- Fork 0
add expo doctor #41
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
base: main
Are you sure you want to change the base?
add expo doctor #41
Conversation
| WalkthroughCI workflows were updated: expo-doctor workflow now runs on push and pull_request for main and uses Node.js 20. The pull-request workflow was simplified to standard steps (checkout, Node 20, yarn install, expo-doctor). package.json version was incremented from 0.0.4 to 0.0.5. Changes
 Sequence Diagram(s)sequenceDiagram
    autonumber
    participant GH as GitHub
    participant AR as Actions Runner
    participant Repo as Repo
    participant Node as setup-node@v4 (Node 20)
    participant Y as yarn
    participant ED as expo-doctor
    GH->>AR: pull_request event [activated]
    AR->>Repo: actions/checkout
    AR->>Node: Setup Node.js v20
    AR->>Y: Install deps (yarn install)
    AR->>ED: Run expo-doctor
    ED-->>AR: Report findings (success/failure)
    Note over AR,ED: Streamlined steps replacing custom scripts [Changed]
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
 Pre-merge checks and finishing touches✅ Passed checks (3 passed)
 ✨ Finishing touches🧪 Generate unit tests (beta)
 Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment  | 
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.
Actionable comments posted: 0
🧹 Nitpick comments (1)
.github/workflows/pull-request.yml (1)
17-21: Use the Yarn script for consistency.Since the repo standardizes on Yarn (see the
doctorscript in package.json), prefer callingyarn doctorinstead of invokingnpx expo-doctordirectly to ensure we keep behavior centralized in the script.- - name: Run expo-doctor - run: npx expo-doctor + - name: Run expo-doctor + run: yarn doctor
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
- .github/workflows/expo-doctor.yml(2 hunks)
- .github/workflows/pull-request.yml(1 hunks)
- package.json(2 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
package.json
📄 CodeRabbit inference engine (.cursor/rules/troubleshooting.mdc)
Use
yarnas the project package manager (as configured in package.json)
Files:
- package.json
🧠 Learnings (1)
📚 Learning: 2025-09-25T10:42:32.852Z
Learnt from: CR
PR: saigontechnology/rn-base-project-expo#0
File: .cursor/rules/troubleshooting.mdc:0-0
Timestamp: 2025-09-25T10:42:32.852Z
Learning: Applies to package.json : Use `yarn` as the project package manager (as configured in package.json)
Applied to files:
- package.json
Summary by CodeRabbit