Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Do not merge: destination-duckdb CDK bump research #52709

Closed
wants to merge 6 commits into from
Closed

Do not merge: destination-duckdb CDK bump research #52709

wants to merge 6 commits into from

Conversation

devin-ai-integration[bot]
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot commented Feb 1, 2025 •
edited by aaronsteers
Loading

Requested by AJ.

Link to Devin run: https://app.devin.ai/sessions/da13ba30ddbf42e4941604ab2fd4aa19
Requested by: Aaron

This PR updates destination-duckdb to use the latest CDK version (6.31.1). Changes:

  1. Bump airbyte-cdk from 0.51.6 to 6.31.1
  2. Update connector version to 0.5.0 to reflect major CDK version change

The core destination interface remains stable across CDK versions, so no code changes were required.

@devin-ai-integration Devin.ai Integration
Copy link
Contributor Author

🤖 Devin AI Engineer

Original prompt from Aaron:

@Devin - Can you bump source-faker to the latest version of the CDK and remove dependencies on Pendulum (if any)? If necessary, please also bump the max python version to 3.12 (<3.12).

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

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@vercel Vercel
Copy link

vercel bot commented Feb 1, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
airbyte-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 1, 2025 4:27am

@octavia-squidington-iii octavia-squidington-iii added the area/connectors Connector related issues label Feb 1, 2025
@devin-ai-integration Devin.ai Integration
Copy link
Contributor Author

/bump-version type=patch

devin-ai-integration bot and others added 2 commits February 1, 2025 03:55
@devin-ai-integration @aaronsteers
fix: Fix type errors in source-faker
767ec6f 
Co-Authored-By: Aaron <AJ> Steers <[email protected]>
@devin-ai-integration Devin.ai Integration
Copy link
Contributor Author

Closing due to inactivity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
area/connectors Connector related issues connectors/destination/duckdb connectors/source/faker
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@marcosmarxm @octavia-squidington-iii