Skip to content

Connector: Verify support for DuckDB 2.x - #24

Merged
amotl merged 1 commit into
mainfrom
duckdb2
Oct 21, 2026
Merged

amotl merged 1 commit into
mainfrom
duckdb2

Conversation

@amotl

@amotl amotl commented Oct 21, 2026

Copy link
Copy Markdown
Member

About

Adjust helper function for compatibility with DuckDB 2.x.

References

@amotl amotl added the dependencies Updates to package dependencies or such. label Oct 21, 2026
@coderabbitai

coderabbitai Bot commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 5a03a99b-b319-408a-9598-b5ceaf5d9aed
📥 Commits

Reviewing files that changed from the base of the PR and between 233447d and 82c66e3.

📒 Files selected for processing: 2
  • pyproject.toml
  • src/dlt_filesystem/source/format/helpers.py
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@amotl amotl linked an issue Oct 21, 2026 that may be closed by this pull request
@amotl
amotl marked this pull request as ready for review October 21, 2026 22:57
@amotl
amotl merged commit 0c57a5a into main Oct 21, 2026
5 of 6 checks passed
@amotl
amotl deleted the duckdb2 branch October 21, 2026 22:58
@codecov

codecov Bot commented Oct 21, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 75.00000% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/dlt_filesystem/source/format/helpers.py 75.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Updates to package dependencies or such.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Compatibility with duckdb 2.x

1 participant