Skip to content

Conversation

@redox
Copy link
Contributor

@redox redox commented Sep 21, 2025

Following duckdb/duckdb#17772 - we need to change the way the extensions are initialized. This fixes the compilation of the extension with DuckDB's current HEAD.

@redox redox requested a review from hatamiarash7 as a code owner September 21, 2025 20:46
@utay
Copy link

utay commented Sep 22, 2025

Fix #10

@hatamiarash7
Copy link
Owner

Thanks for the contribution ❤️ .
I think you forgot to update the duckdb submodule, so the tests failed. I will do that before release.

@hatamiarash7 hatamiarash7 merged commit 50860a7 into hatamiarash7:main Sep 22, 2025
11 of 24 checks passed
@redox
Copy link
Contributor Author

redox commented Sep 22, 2025

I think you forgot to update the duckdb submodule, so the tests failed. I will do that before release.

Ah gosh, thank you - I'll check how you're proceeding to do this.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants