Skip to content

chore: cleaning up warnings and preparing 0.29.3#3910

Merged
rtyler merged 2 commits intodelta-io:mainfrom
rtyler:chore/cleanup-warnings-before-0.29.3
Oct 31, 2025
Merged

chore: cleaning up warnings and preparing 0.29.3#3910
rtyler merged 2 commits intodelta-io:mainfrom
rtyler:chore/cleanup-warnings-before-0.29.3

Conversation

@rtyler
Copy link
Copy Markdown
Member

@rtyler rtyler commented Oct 31, 2025

  • chore: remove deprecated warnings on pub(crate) and a few others
  • chore: grab-bag of compiler warning remediations
  • chore: grab bag of warnings removals

We should not be littering the logs with warnings until we have an
immediate plan
to remove the deprecated functions. `new_metadata` is not able to be
replaced until an actual replacement with kernel is available. At that
time we can mark this as deprecated

Signed-off-by: R. Tyler Croy <rtyler@brokenco.de>
@github-actions github-actions bot added the binding/rust Issues for the Rust crate label Oct 31, 2025
@codecov
Copy link
Copy Markdown

codecov bot commented Oct 31, 2025

Codecov Report

❌ Patch coverage is 84.19958% with 76 lines in your changes missing coverage. Please review.
✅ Project coverage is 73.87%. Comparing base (e9028cb) to head (e73159f).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
crates/core/src/protocol/checkpoints.rs 72.37% 65 Missing and 6 partials ⚠️
crates/core/src/table/state.rs 0.00% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3910      +/-   ##
==========================================
- Coverage   73.89%   73.87%   -0.03%     
==========================================
  Files         152      152              
  Lines       39415    39404      -11     
  Branches    39415    39404      -11     
==========================================
- Hits        29125    29108      -17     
- Misses       8966     8973       +7     
+ Partials     1324     1323       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Signed-off-by: R. Tyler Croy <rtyler@brokenco.de>
@rtyler rtyler force-pushed the chore/cleanup-warnings-before-0.29.3 branch from ed8c1b2 to e73159f Compare October 31, 2025 13:01
@rtyler rtyler marked this pull request as ready for review October 31, 2025 13:12
@rtyler rtyler requested review from hntd187 and roeap as code owners October 31, 2025 13:12
@rtyler rtyler enabled auto-merge (rebase) October 31, 2025 13:12
@rtyler rtyler merged commit 2e5f7b0 into delta-io:main Oct 31, 2025
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

binding/rust Issues for the Rust crate

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants