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

Clippy (elided lifetimes) + rustfmt #956

Merged
merged 3 commits into from
Dec 1, 2024
Merged

Conversation

Bromeon
Copy link
Member

@Bromeon Bromeon commented Dec 1, 2024

Follow clippy lints which seem to be newly enforced as of Rust 1.83:

  • clippy::needless-lifetimes
  • clippy::elided-named-lifetimes
  • clippy::empty-line-after-doc-comments

Also, rustfmt requires changes in several parts.

Also applies clippy::empty-line-after-doc-comments in 2 instances, one of which
was an accidental /// comment instead of //!.
@Bromeon Bromeon added quality-of-life No new functionality, but improves ergonomics/internals c: tooling CI, automation, tools labels Dec 1, 2024
@Bromeon Bromeon enabled auto-merge December 1, 2024 16:43
@Bromeon Bromeon added this pull request to the merge queue Dec 1, 2024
@GodotRust
Copy link

API docs are being generated and will be shortly available at: https://godot-rust.github.io/docs/gdext/pr-956

Merged via the queue into master with commit 385a765 Dec 1, 2024
15 checks passed
@Bromeon Bromeon deleted the qol/clippy-elided-lifetimes branch December 1, 2024 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c: tooling CI, automation, tools quality-of-life No new functionality, but improves ergonomics/internals
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants