chore: update dependencies for mcp compatability#799
chore: update dependencies for mcp compatability#799Patrick Nilan (pnilan) wants to merge 3 commits intomainfrom
Conversation
|
Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. 📝 WalkthroughWalkthroughUpdated dependency constraints in Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Suggested labels
Have you verified compatibility (local tests/CI) with Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (10)
Comment |
There was a problem hiding this comment.
Pull Request Overview
This PR addresses dependency version updates to maintain compatibility and stability: bumping jsonref from ~0.2 to ^1 (1.1.0) for airbyte-cdk import compatibility, and reverting ddtrace from ^3.12.3 to 3.12.0 due to the newer version being yanked from PyPI.
Key changes:
- Updated
jsonrefto version 1.x with breaking changes confirmed not to affect airbyte-cdk usage - Reverted
ddtraceto 3.12.0 and removed theoptional = trueflag
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
👋 Greetings, Airbyte Team Member!Here are some helpful tips and reminders for your convenience. Testing This CDK VersionYou can test this version of the CDK using the following: # Run the CLI from this branch:
uvx 'git+https://github.com/airbytehq/airbyte-python-cdk.git@pnilan/jsonref/bump-version#egg=airbyte-python-cdk[dev]' --help
# Update a connector to use the CDK from this branch ref:
cd airbyte-integrations/connectors/source-example
poe use-cdk-branch pnilan/jsonref/bump-versionHelpful ResourcesPR Slash CommandsAirbyte Maintainers can execute the following slash commands on your PR:
|
PyTest Results (Fast)1 tests - 3 801 0 ✅ - 3 790 1s ⏱️ - 6m 31s For more details on these errors, see this check. Results for commit dadb972. ± Comparison against base commit 55ea028. This pull request removes 3802 and adds 1 tests. Note that renamed tests count towards both.♻️ This comment has been updated with latest results. |
PyTest Results (Full)3 tests - 3 802 0 ✅ - 3 793 16s ⏱️ - 10m 59s For more details on these errors, see this check. Results for commit dadb972. ± Comparison against base commit 55ea028. This pull request removes 3805 and adds 3 tests. Note that renamed tests count towards both.♻️ This comment has been updated with latest results. |
What
airbyte-cdkto the mcp repo, I'm updating thejsonrefdep to 1.1.0, the breaking changes do not affect its use in the airbyte-cdk packageddtrace3.12.3 was yanked due to missing commits that were in 3.12.0, therefore, reverting to 3.12.0numpyto2.4.0.langchainandlangchain-core.Summary by CodeRabbit