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: Switch to once_cell #3829

Merged
merged 1 commit into from
Jun 14, 2022
Merged

refactor: Switch to once_cell #3829

merged 1 commit into from
Jun 14, 2022

Conversation

epage
Copy link
Member

@epage epage commented Jun 14, 2022

Though this is changing an API item we export, we do not consider this a
breaking change because

  • This was an implementation detail of the macros and people shouldn't be using it directly
  • The macro_rules macro is coupled to clap because they are in the
    same crate
  • The derive macro is coupled to clap because clap declares a
    =x.y.z dependency on `clap_derive

Fixes #3828

Though this is changing an API item we export, we do not consider this a
breaking change because
- This was an implementation detail of the macros and people shouldn't be using it directly
- The `macro_rules` macro is coupled to `clap` because they are in the
  same crate
- The derive macro is coupled to `clap` because `clap` declares a
  `=x.y.z` dependency on `clap_derive

Fixes clap-rs#3828
@epage epage merged commit d71e38e into clap-rs:master Jun 14, 2022
@epage epage deleted the lazy branch June 14, 2022 14:44
@omjadas omjadas mentioned this pull request Jun 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Replace lazy_static with once_cell
1 participant