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: fix new warnings in Rust 1.72.0 #2700

Merged
merged 6 commits into from
Aug 29, 2023
Merged

chore: fix new warnings in Rust 1.72.0 #2700

merged 6 commits into from
Aug 29, 2023

Conversation

hawkw
Copy link
Member

@hawkw hawkw commented Aug 29, 2023

This branch fixes a handful of new warnings which have shown up after
updating to Rust 1.72.0.

This includes:

  • clippy::redundant_closure_call in macros --- allowed because the
    macro sometimes calls a function that isn't a closure, and the closure
    is just used in the case where it's not a function.
  • Unnecessary uses of # in raw string literals that don't contain "
    characters.
  • Dead code warnings with specific feature flag combinations in
    tracing-subscriber.

In addition, I've fixed a broken RustDoc link that was making the
Netlify build sad.

@hawkw hawkw requested review from carllerche, davidbarsky and a team as code owners August 29, 2023 00:13
@hawkw hawkw enabled auto-merge (squash) August 29, 2023 00:13
@hawkw hawkw disabled auto-merge August 29, 2023 00:17
@hawkw hawkw enabled auto-merge (squash) August 29, 2023 00:17
@hawkw hawkw merged commit bf05c61 into master Aug 29, 2023
57 checks passed
@hawkw hawkw deleted the eliza/dewarn branch August 29, 2023 18:17
davidbarsky pushed a commit that referenced this pull request Sep 26, 2023
This branch fixes a handful of new warnings which have shown up after
updating to Rust 1.72.0.

This includes:

* `clippy::redundant_closure_call` in macros --- allowed because the
  macro sometimes calls a function that isn't a closure, and the closure
  is just used in the case where it's not a function.
* Unnecessary uses of `#` in raw string literals that don't contain `"`
  characters.
* Dead code warnings with specific feature flag combinations in
  `tracing-subscriber`.

In addition, I've fixed a broken RustDoc link that was making the
Netlify build sad.
davidbarsky pushed a commit that referenced this pull request Sep 27, 2023
This branch fixes a handful of new warnings which have shown up after
updating to Rust 1.72.0.

This includes:

* `clippy::redundant_closure_call` in macros --- allowed because the
  macro sometimes calls a function that isn't a closure, and the closure
  is just used in the case where it's not a function.
* Unnecessary uses of `#` in raw string literals that don't contain `"`
  characters.
* Dead code warnings with specific feature flag combinations in
  `tracing-subscriber`.

In addition, I've fixed a broken RustDoc link that was making the
Netlify build sad.
davidbarsky pushed a commit that referenced this pull request Sep 27, 2023
This branch fixes a handful of new warnings which have shown up after
updating to Rust 1.72.0.

This includes:

* `clippy::redundant_closure_call` in macros --- allowed because the
  macro sometimes calls a function that isn't a closure, and the closure
  is just used in the case where it's not a function.
* Unnecessary uses of `#` in raw string literals that don't contain `"`
  characters.
* Dead code warnings with specific feature flag combinations in
  `tracing-subscriber`.

In addition, I've fixed a broken RustDoc link that was making the
Netlify build sad.
davidbarsky pushed a commit that referenced this pull request Sep 27, 2023
This branch fixes a handful of new warnings which have shown up after
updating to Rust 1.72.0.

This includes:

* `clippy::redundant_closure_call` in macros --- allowed because the
  macro sometimes calls a function that isn't a closure, and the closure
  is just used in the case where it's not a function.
* Unnecessary uses of `#` in raw string literals that don't contain `"`
  characters.
* Dead code warnings with specific feature flag combinations in
  `tracing-subscriber`.

In addition, I've fixed a broken RustDoc link that was making the
Netlify build sad.
davidbarsky pushed a commit that referenced this pull request Sep 27, 2023
This branch fixes a handful of new warnings which have shown up after
updating to Rust 1.72.0.

This includes:

* `clippy::redundant_closure_call` in macros --- allowed because the
  macro sometimes calls a function that isn't a closure, and the closure
  is just used in the case where it's not a function.
* Unnecessary uses of `#` in raw string literals that don't contain `"`
  characters.
* Dead code warnings with specific feature flag combinations in
  `tracing-subscriber`.

In addition, I've fixed a broken RustDoc link that was making the
Netlify build sad.
davidbarsky pushed a commit that referenced this pull request Sep 29, 2023
This branch fixes a handful of new warnings which have shown up after
updating to Rust 1.72.0.

This includes:

* `clippy::redundant_closure_call` in macros --- allowed because the
  macro sometimes calls a function that isn't a closure, and the closure
  is just used in the case where it's not a function.
* Unnecessary uses of `#` in raw string literals that don't contain `"`
  characters.
* Dead code warnings with specific feature flag combinations in
  `tracing-subscriber`.

In addition, I've fixed a broken RustDoc link that was making the
Netlify build sad.
hawkw added a commit that referenced this pull request Oct 1, 2023
This branch fixes a handful of new warnings which have shown up after
updating to Rust 1.72.0.

This includes:

* `clippy::redundant_closure_call` in macros --- allowed because the
  macro sometimes calls a function that isn't a closure, and the closure
  is just used in the case where it's not a function.
* Unnecessary uses of `#` in raw string literals that don't contain `"`
  characters.
* Dead code warnings with specific feature flag combinations in
  `tracing-subscriber`.

In addition, I've fixed a broken RustDoc link that was making the
Netlify build sad.
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.

2 participants