Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: remove lazy_static and once_cell dependencies #3148

Merged
merged 1 commit into from
Jan 21, 2025

Conversation

nyurik
Copy link
Contributor

@nyurik nyurik commented Jan 20, 2025

Description

LazyLock has been available since 1.80 which is less than MSRV

@github-actions github-actions bot added binding/python Issues for the Python package binding/rust Issues for the Rust crate labels Jan 20, 2025
@nyurik nyurik force-pushed the remove-lazy-static branch from b1accc7 to 9e86284 Compare January 20, 2025 23:34
Copy link

codecov bot commented Jan 21, 2025

Codecov Report

Attention: Patch coverage is 89.25620% with 26 lines in your changes missing coverage. Please review.

Project coverage is 72.15%. Comparing base (82ef6bf) to head (433eb97).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
crates/core/src/kernel/models/fields.rs 80.64% 12 Missing ⚠️
crates/aws/src/constants.rs 0.00% 8 Missing ⚠️
crates/azure/src/config.rs 0.00% 3 Missing ⚠️
crates/gcp/src/config.rs 0.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3148      +/-   ##
==========================================
+ Coverage   71.96%   72.15%   +0.19%     
==========================================
  Files         135      137       +2     
  Lines       43581    44082     +501     
  Branches    43581    44082     +501     
==========================================
+ Hits        31361    31808     +447     
- Misses      10181    10243      +62     
+ Partials     2039     2031       -8     

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

@nyurik nyurik force-pushed the remove-lazy-static branch from 9e86284 to 561c52e Compare January 21, 2025 00:10
@nyurik nyurik changed the title chore: remove lazy_static dep in favor of LazyLock chore: remove lazy_static and once_cell dependencies Jan 21, 2025
roeap
roeap previously approved these changes Jan 21, 2025
Copy link
Collaborator

@roeap roeap left a comment

Choose a reason for hiding this comment

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

Thanks for taking care of this!! Always great to get rid of dependencies ❤️.

@roeap roeap enabled auto-merge January 21, 2025 09:01
@roeap
Copy link
Collaborator

roeap commented Jan 21, 2025

@nyurik - looks like we need one more cargo fmt :).

LazyLock has been available since 1.80 which is less than MSRV

Signed-off-by: Yuri Astrakhan <[email protected]>
@rtyler rtyler added this to the v0.24 milestone Jan 21, 2025
@roeap roeap added this pull request to the merge queue Jan 21, 2025
Merged via the queue into delta-io:main with commit a2cc2ba Jan 21, 2025
26 checks passed
@nyurik nyurik deleted the remove-lazy-static branch January 21, 2025 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
binding/python Issues for the Python package binding/rust Issues for the Rust crate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants