-
Notifications
You must be signed in to change notification settings - Fork 130
feat: allow for linking DuckDB debug build #4705
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
Conversation
33916d2 to
02c23cc
Compare
Signed-off-by: Alexander Droste <alexander.droste@protonmail.com>
02c23cc to
71a1986
Compare
Codecov Report❌ Patch coverage is
☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
1ec4d67 to
71a1986
Compare
Deploying vortex-bench with
|
| Latest commit: |
71a1986
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://3d7a589e.vortex-93b.pages.dev |
| Branch Preview URL: | https://ad-duckdb-debug.vortex-93b.pages.dev |
connortsui20
left a comment
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.
It looks fine to me, but I think it would be best if someone more familiar with DuckDB builds looks at this? Also don't really have time to test this locally myself
|
oh it merged haha |
|
I think we want to run tests with debug build so we get all the assertions |
Yes. |
No description provided.