-
Notifications
You must be signed in to change notification settings - Fork 62
chore: integration test target fix, reverse sync py-sdk #671
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
Conversation
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the 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. WalkthroughThis pull request separates unit and integration tests for the Python client by excluding integration tests from the default test run with a pytest filter, adds a dedicated integration test target to the Makefile, and bumps the Python SDK version to 0.9.9. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3–5 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
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.
Pull request overview
This PR performs a reverse sync from the Python SDK repository and adds proper integration test handling to the SDK generator. The changes align the Python SDK configuration with recent releases and separate integration tests from unit tests in the build process.
Key Changes:
- Bumps Python SDK version from 0.9.7 to 0.9.9
- Separates integration and unit tests in the Python test target using pytest markers
- Adds a dedicated integration test target for Python
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| config/clients/python/config.overrides.json | Updates package version from 0.9.7 to 0.9.9 to sync with Python SDK releases |
| Makefile | Modifies test-client-python to exclude integration tests and adds test-integration-client-python target to run integration tests separately |
Co-authored-by: Copilot <[email protected]>
Description
What problem is being solved?
How is it being solved?
What changes are made to solve it?
References
Review Checklist
mainSummary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.