Skip to content

Conversation

Cleboost
Copy link
Contributor

Replaces instances of format!(...) with positional arguments (e.g., format!("{}", var)) to the new Rust 1.58+ inline variable syntax (e.g., format!("{var}")). This change improves code readability and consistency across the codebase, and leverages modern Rust formatting features. No functional changes are introduced.

This fix Github Action Code style. (Not sure this is usefull)

Replaces instances of format!(...) with positional arguments (e.g., format!("{}", var)) to the new Rust 1.58+ inline variable syntax (e.g., format!("{var}")). This change improves code readability and consistency across the codebase, and leverages modern Rust formatting features. No functional changes are introduced.
@jacksongoode
Copy link
Collaborator

I think this is good, hoping the macOS build passes.

Copy link
Collaborator

@jacksongoode jacksongoode left a comment

Choose a reason for hiding this comment

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

Looks good, thanks for being so quick to this!

@Cleboost
Copy link
Contributor Author

I will check tomorrow for mac os build

@jacksongoode
Copy link
Collaborator

For some reason we now need the --skip-jenkins flag on create-dmg.

@jacksongoode jacksongoode merged commit 35e28a6 into jpochyla:main Jul 15, 2025
5 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.

2 participants