Skip to content

Conversation

@albertlockett
Copy link
Member

supersedes #1822 which was failing on a cargo deny license check for an optional dependency (bzip2)

@albertlockett albertlockett requested a review from a team as a code owner January 19, 2026 18:06
@github-actions github-actions bot added the rust Pull requests that update Rust code label Jan 19, 2026
@albertlockett albertlockett changed the title udpate zip crate to 7.1.0 chore(deps): update zip crate to 7.1.0 Jan 19, 2026
@codecov
Copy link

codecov bot commented Jan 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.70%. Comparing base (30e7b3e) to head (ca89a78).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1825   +/-   ##
=======================================
  Coverage   84.70%   84.70%           
=======================================
  Files         498      498           
  Lines      146255   146255           
=======================================
+ Hits       123879   123881    +2     
+ Misses      21842    21840    -2     
  Partials      534      534           
Components Coverage Δ
otap-dataflow 86.13% <ø> (+<0.01%) ⬆️
query_abstraction 80.61% <ø> (ø)
query_engine 90.52% <ø> (ø)
syslog_cef_receivers ∅ <ø> (∅)
otel-arrow-go 53.50% <ø> (ø)
quiver 90.66% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@albertlockett
Copy link
Member Author

ugh, looks like this is still failing the cargo deny check b/c we run this with --all-features which will include the bzip feature on the zip crate. I'll come back to this later.

@albertlockett albertlockett marked this pull request as draft January 20, 2026 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rust Pull requests that update Rust code

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant