Fix: resolve gql dependency error by pinning version in mcp_rl (Fixes…#248
Fix: resolve gql dependency error by pinning version in mcp_rl (Fixes…#248CtrlAltDefeat21 wants to merge 1 commit into
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe notebook's dependency installation cell is updated to include ChangesNotebook Dependency Fix
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 Ruff (0.15.20)art_mcp_rl/mcp_rl.ipynbUnexpected end of JSON input 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 |
Description
This PR resolves the
ImportError: cannot import name 'TransportConnectionFailed'that occurs when running themcp_rl.ipynbnotebook in Google Colab.The issue stems from a dependency mismatch where
weaveexpectsTransportConnectionFailedto be exposed, but the environment pulls an incompatible beta version ofgql(3.6.0b4).Changes Made
mcp_rl.ipynbto explicitly pingql>=4.0.0usinguv pip installbefore other dependencies are pulled.Fixes #234
Summary by CodeRabbit