fix: remove duplicate @expo/metro-config override (unbreak vp install / Release Desktop)#107
Conversation
The fork already pins @expo/metro-config to 56.0.13 (182070b). The pingdotgg#3041 cherry-pick re-added an identical override key, producing a duplicate mapping key that broke "vp install" YAML parsing (CI + Release Desktop preflight).
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughRemoves two entries from the Changespnpm workspace override cleanup
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
✨ 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 |
vp installwas failing withERROR duplicated mapping key (66:3), breaking CI and the Release Desktop preflight.The fork pins
@expo/metro-config: 56.0.13inoverrides:(182070b). The pingdotgg#3041 cherry-pick re-added an identical override key, creating a duplicate mapping key that pnpm's strict YAML parser rejects. Removed the duplicate (kept the fork's entry).\n\nVerified: no duplicate mapping keys remain inpnpm-workspace.yaml.Summary by CodeRabbit