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

refactor: Replace lazy_static with std::sync::LazyLock #827

Merged
merged 9 commits into from
Jul 31, 2024

Conversation

Techassi
Copy link
Member

@Techassi Techassi commented Jul 25, 2024

std::cell::LazyCell and std::sync::LazyLock were stabilized in Rust 1.80.0. We can now use the standard library types instead of the lazy_static crate.

This change is marked as breaking, because it requires downstream operators to bump their Rust version to 1.80.0 to support the new stabilized standard library type.

Reviewer

std::cell::LazyCell and std::sync::LazyLock were
stabilized in Rust 1.80.0. We can now use the
standard library types instead of the lazy_static
crate.
@Techassi Techassi self-assigned this Jul 25, 2024
Copy link
Member

@NickLarsenNZ NickLarsenNZ left a comment

Choose a reason for hiding this comment

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

Just the PR number needs fixing

crates/k8s-version/CHANGELOG.md Outdated Show resolved Hide resolved
crates/stackable-operator/src/kvp/key.rs Show resolved Hide resolved
Co-authored-by: Nick <10092581+NickLarsenNZ@users.noreply.github.com>
NickLarsenNZ
NickLarsenNZ previously approved these changes Jul 31, 2024
Copy link
Member

@NickLarsenNZ NickLarsenNZ left a comment

Choose a reason for hiding this comment

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

LGTM

@Techassi Techassi added this pull request to the merge queue Jul 31, 2024
@Techassi Techassi removed this pull request from the merge queue due to a manual request Jul 31, 2024
@Techassi Techassi added this pull request to the merge queue Jul 31, 2024
Merged via the queue into main with commit f15a9a3 Jul 31, 2024
20 checks passed
@Techassi Techassi deleted the refactor/replace-lazy_static branch July 31, 2024 08:28
nightkr added a commit that referenced this pull request Aug 8, 2024
This reverts commit f15a9a3.

We still need Rust 1.79.0 compatibility until 1.80 percolates into Nixpkgs.
github-merge-queue bot pushed a commit that referenced this pull request Aug 9, 2024
#835)

* Revert "refactor: Replace lazy_static with std::sync::LazyLock (#827)"

This reverts commit f15a9a3.

We still need Rust 1.79.0 compatibility until 1.80 percolates into Nixpkgs.

* Changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants