Skip to content

Conversation

@ucswift
Copy link
Member

@ucswift ucswift commented Jan 20, 2026

Summary by CodeRabbit

  • Chores
    • Updated build configuration to include additional files in container builds for improved build context management.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 20, 2026

📝 Walkthrough

Walkthrough

The .dockerignore file was modified to include the __mocks__/ directory in Docker builds, removing its previous exclusion. An explanatory comment was added to document that metro.config.js requires these mocks for web builds.

Changes

Cohort / File(s) Summary
Configuration
\.dockerignore
Removed __mocks__/ exclusion and added comment clarifying that this directory is needed for metro.config.js web builds

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A docker build once spurned the mock,
But metro said "Wait, stop the clock!"
For web builds need these files dear,
So now they're invited to appear. 🎭

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'RD-T39 Fixing build' is vague and generic. While it references a ticket identifier (RD-T39), the term 'Fixing build' is too broad and does not clearly convey the specific change made (modifying .dockerignore to include mocks for metro.config.js web builds). Consider using a more descriptive title that specifically mentions the change, such as 'Include mocks in Docker build context for metro.config.js' to better communicate the actual modification.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ucswift
Copy link
Member Author

ucswift commented Jan 20, 2026

Approve

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR is approved.

@ucswift ucswift merged commit bfbc163 into master Jan 20, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants