duckdb / duckdb_spatial

MIT License
492 stars 41 forks source link

duckdb-wasm: Use -sSIDE_MODULE=2 #234

Closed carlopi closed 10 months ago

carlopi commented 10 months ago

Bumped against this, this reduces wasm total size by about 10%. Tested locally.

I will also add this to the main duckdb/duckdb-wasm pipeline, but those have to be independent PRs.

Maxxen commented 10 months ago

@carlopi Happy to merge this, but any idea what's up with the CI here?

carlopi commented 10 months ago

@carlopi Happy to merge this, but any idea what's up with the CI here?

No idea. Maybe can you restart the workflow?