Skip to content

feat(typed-builder): adopt typed-builder for safer builder pattern in non-core crates#3902

Merged
rtyler merged 4 commits intodelta-io:mainfrom
fvaleye:feature/typed-builder-first-round
Oct 30, 2025
Merged

feat(typed-builder): adopt typed-builder for safer builder pattern in non-core crates#3902
rtyler merged 4 commits intodelta-io:mainfrom
fvaleye:feature/typed-builder-first-round

Conversation

@fvaleye
Copy link
Copy Markdown
Collaborator

@fvaleye fvaleye commented Oct 28, 2025

Description

This PR refactors existing builder implementations to use the typed-builder crate.
This is the first phase of the migration, covering all "non-core crates".

The core crate (the largest and most complex one) will be migrated in a follow-up PR.

Documentation

@github-actions github-actions bot added the binding/rust Issues for the Rust crate label Oct 28, 2025
@codecov
Copy link
Copy Markdown

codecov bot commented Oct 28, 2025

Codecov Report

❌ Patch coverage is 92.20779% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 73.87%. Comparing base (5ac0629) to head (772a91a).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
crates/aws/src/lib.rs 88.46% 2 Missing and 1 partial ⚠️
crates/catalog-unity/src/lib.rs 86.95% 1 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3902      +/-   ##
==========================================
+ Coverage   73.76%   73.87%   +0.10%     
==========================================
  Files         152      152              
  Lines       39524    39393     -131     
  Branches    39524    39393     -131     
==========================================
- Hits        29156    29102      -54     
+ Misses       9044     8968      -76     
+ 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: Florian Valeye <florian.valeye@gmail.com>
…nity crate

Signed-off-by: Florian Valeye <florian.valeye@gmail.com>
@fvaleye fvaleye force-pushed the feature/typed-builder-first-round branch from 529b4f7 to f4e462d Compare October 28, 2025 15:52
@rtyler rtyler merged commit e9028cb into delta-io:main Oct 30, 2025
35 of 36 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