Skip to content

Conversation

@htiennv
Copy link
Contributor

@htiennv htiennv commented Sep 24, 2025

This commit removes the once_cell dependency from Cargo.toml files in the codex-rs and apply-patch directories, replacing its usage with std::sync::LazyLock and std::sync::OnceLock where applicable. This change simplifies the dependency tree and utilizes standard library features for lazy initialization.

External (non-OpenAI) Pull Request Requirements

Before opening this Pull Request, please read the dedicated "Contributing" markdown file or your PR may be closed:
https://github.com/openai/codex/blob/main/docs/contributing.md

If your PR conforms to our contribution guidelines, replace this text with a detailed and high quality description of your changes.

This commit removes the `once_cell` dependency from `Cargo.toml` files in the `codex-rs` and `apply-patch` directories, replacing its usage with `std::sync::LazyLock` and `std::sync::OnceLock` where applicable. This change simplifies the dependency tree and utilizes standard library features for lazy initialization.
@htiennv
Copy link
Contributor Author

htiennv commented Sep 24, 2025

I have read the CLA Document and I hereby sign the CLA

@github-actions
Copy link
Contributor

github-actions bot commented Sep 24, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

github-actions bot added a commit that referenced this pull request Sep 24, 2025
@bolinfest bolinfest requested a review from jif-oai September 24, 2025 15:32
@bolinfest bolinfest merged commit 00c7f7a into openai:main Sep 24, 2025
19 of 20 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Sep 24, 2025
@htiennv htiennv deleted the feat/remove-oncecell branch September 24, 2025 16:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants