Skip to content

refactor: upgrade Rust edition to 2024#221

Merged
mkqavi merged 1 commit intomainfrom
push-lyyxozmsrnym
May 4, 2026
Merged

refactor: upgrade Rust edition to 2024#221
mkqavi merged 1 commit intomainfrom
push-lyyxozmsrnym

Conversation

@mkqavi
Copy link
Copy Markdown
Owner

@mkqavi mkqavi commented May 4, 2026

No description provided.

@mkqavi mkqavi self-assigned this May 4, 2026
@mkqavi mkqavi requested a review from Copilot May 4, 2026 07:01
Copy link
Copy Markdown

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 crate to Rust 2024 edition and applies the small source-level rewrites needed for that edition/style refresh. In the broader codebase, these changes primarily affect build/toolchain compatibility rather than application behavior.

Changes:

  • Switch the crate manifest from Rust 2021 to Rust 2024.
  • Apply rustfmt-style import reordering in several modules.
  • Simplify one nested if in main.rs into a chained if let expression and keep formatting/parsing-compatible cleanup in config.rs.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
Cargo.toml Changes the package edition to 2024.
src/main.rs Reorders imports and rewrites one repo-ignore check using if let chaining.
src/scp.rs Reorders anyhow imports only.
src/repo.rs Reorders crate imports only.
src/config.rs Applies a formatting-only semicolon/parenthesis adjustment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread Cargo.toml
Comment thread src/main.rs
@mkqavi mkqavi merged commit 63e5048 into main May 4, 2026
7 checks passed
@mkqavi mkqavi deleted the push-lyyxozmsrnym branch May 4, 2026 07:07
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