Skip to content

[rust] Update code to support 1.83 in clippy#5

Merged
gregnazario merged 2 commits intomainfrom
update-to-1.83-compatible
Apr 24, 2025
Merged

[rust] Update code to support 1.83 in clippy#5
gregnazario merged 2 commits intomainfrom
update-to-1.83-compatible

Conversation

@gregnazario
Copy link
Contributor

No description provided.

@gregnazario gregnazario force-pushed the update-to-1.83-compatible branch from 9d54415 to 9e385bc Compare April 24, 2025 19:13
@gregnazario gregnazario requested a review from Copilot April 24, 2025 19:13
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the codebase to support Rust 1.83 in clippy by adjusting lint attributes, lifetime annotations, and CI tooling versions. Key changes include:

  • Adjusting lint allow attributes in tests and source files (e.g. replacing unit_arg with new Clippy lints).
  • Updating lifetime parameters in various Serialize/Deserialize trait implementations to use underscore lifetimes.
  • Upgrading GitHub Actions versions and toolchain settings in CI workflows.

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tests/serde.rs Added a new lint allowance and updated constant usage from u64::max_value() to u64::MAX.
src/ser.rs Replaced explicit lifetime parameters with underscore lifetimes in serializer trait implementations.
src/de.rs Simplified lifetime declarations and updated Clippy lint allowances in deserializer implementations.
.github/workflows/rustdoc.yml Upgraded GitHub Actions versions for repository checkout and docs deployment.
.github/workflows/ci.yml Upgraded checkout action, updated toolchain version, and refined the clippy command to skip deps.

@gregnazario gregnazario merged commit 174354d into main Apr 24, 2025
4 checks passed
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.

1 participant