Skip to content

chore: remove embedded module#380

Merged
Natik Gadzhi (natikgadzhi) merged 2 commits intomainfrom
devin/1741115435-remove-embedded-module
Mar 4, 2025
Merged

chore: remove embedded module#380
Natik Gadzhi (natikgadzhi) merged 2 commits intomainfrom
devin/1741115435-remove-embedded-module

Conversation

@devin-ai-integration
Copy link
Copy Markdown
Contributor

Removes the embedded module from the airbyte-python-cdk repository as requested.

Link to Devin run: https://app.devin.ai/sessions/a062e4002b7940048f4484c184747536

Co-Authored-By: natik@airbyte.io <natik@airbyte.io>
@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

devin-ai-integration bot commented Mar 4, 2025

🤖 Devin AI Engineer

Original prompt from natik@airbyte.io:

Hey @Devin, can you make a PR that removes the `embedded` module of the airbyte-python-cdk please?

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add "(aside)" to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@github-actions github-actions bot added the chore label Mar 4, 2025
Co-Authored-By: natik@airbyte.io <natik@airbyte.io>
@natikgadzhi Natik Gadzhi (natikgadzhi) changed the title chore: remove embedded module (do not merge) chore: remove embedded module Mar 4, 2025
Copy link
Copy Markdown
Contributor

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.

@natikgadzhi Natik Gadzhi (natikgadzhi) marked this pull request as ready for review March 4, 2025 19:31
@natikgadzhi Natik Gadzhi (natikgadzhi) enabled auto-merge (squash) March 4, 2025 19:31
@natikgadzhi
Copy link
Copy Markdown
Contributor

Since this is technically breaking, we should probably do a 7.0 release.

@natikgadzhi Natik Gadzhi (natikgadzhi) merged commit caa24a0 into main Mar 4, 2025
25 checks passed
@natikgadzhi Natik Gadzhi (natikgadzhi) deleted the devin/1741115435-remove-embedded-module branch March 4, 2025 20:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant