-
Notifications
You must be signed in to change notification settings - Fork 6
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
Upgrade duckdb-wasm and apache-arrow #18
Comments
Sorry for delay. I just pushed a new version that makes @duckdb/duckdb-wasm a peer dependency instead. This way you can update the core duckdb version as you like, without blocking on me :) You'll need to I've tested and everything seems to work, but let me know if you have any issues with this change. |
hm, I'm trying to use a recent dev version and seeing this error: npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving: [email protected]
npm ERR! Found: @duckdb/[email protected]
npm ERR! node_modules/@duckdb/duckdb-wasm
npm ERR! @duckdb/duckdb-wasm@"1.28.1-dev271.0" from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer @duckdb/duckdb-wasm@"^1.28.0" from [email protected] I'm not sure why |
Thanks, I pushed a new version, can you try again? Looks like the dev version numbers aren't working with For now, I made the peer dependency versioning very loose:
I think that should be fine, as this library doesn't depend on anything from duckdb-wasm that's likely to break. But let me know if you run into any issues. I tested it works with "1.28.1-dev258.0" (but it looks like newer versions are currently broken) |
Could you please upgrade the duckdb-wasm and apache-arrow? The latest version https://www.npmjs.com/package/@duckdb/duckdb-wasm/v/1.28.1-dev258.0 uses Arrow v17 instead of v15 a well:
https://github.com/duckdb/duckdb-wasm/blob/main/packages/duckdb-wasm/package.json#L26
I can create a PR if you like. Thanks!
The text was updated successfully, but these errors were encountered: