-
Notifications
You must be signed in to change notification settings - Fork 12
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
chore: remove embedded module #380
Conversation
Co-Authored-By: natik@airbyte.io <natik@airbyte.io>
🤖 Devin AI EngineerOriginal prompt from natik@airbyte.io:
I'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
Co-Authored-By: natik@airbyte.io <natik@airbyte.io>
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.
When tests pass, this is fine.
unit_tests/sources/declarative/resolvers/test_config_components_resolver.py
Show resolved
Hide resolved
Since this is technically breaking, we should probably do a 7.0 release. |
* main: ci: add Docker build workflow for PR checks (airbytehq#385) fix: Update DEFAULT_ERROR_MAPPING for InvalidURL to RETRY (airbytehq#384) chore: use python base image 4.0.0 in SDM (airbytehq#382) ci: add check for 'do not merge' in PR titles (airbytehq#383) chore: remove embedded module (airbytehq#380) build: update pyarrow to ^19.0.0 (do not merge) (airbytehq#379)
Removes the embedded module from the airbyte-python-cdk repository as requested.
Link to Devin run: https://app.devin.ai/sessions/a062e4002b7940048f4484c184747536