Skip to content

Conversation

@yiweichi
Copy link
Member

@yiweichi yiweichi commented Aug 4, 2025

This pr imports zstd from da codec instead of have a seperate configuration.

Issue: scroll-tech/da-codec#58

@yiweichi yiweichi changed the title Import zstd from da codec feat: import zstd encoder crate from da-codec repo Aug 4, 2025
@yiweichi yiweichi changed the title feat: import zstd encoder crate from da-codec repo feat: import zstd encoder crate from da-codec repository Aug 4, 2025
@codspeed-hq
Copy link

codspeed-hq bot commented Aug 4, 2025

CodSpeed Performance Report

Merging #303 will not alter performance

Comparing import-zstd-from-da-codec (e915bd9) with scroll (f451fe8)

Summary

✅ 77 untouched benchmarks

lispc
lispc previously approved these changes Aug 5, 2025
@yiweichi yiweichi force-pushed the import-zstd-from-da-codec branch from 63eefc7 to 2da511e Compare August 6, 2025 09:15
Copy link

@greged93 greged93 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@yiweichi yiweichi merged commit e204180 into scroll Aug 6, 2025
44 checks passed
@yiweichi yiweichi deleted the import-zstd-from-da-codec branch August 6, 2025 14:27
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.

5 participants