Skip to content

Simplify slide on branding in action#3089

Merged
hurryabit merged 1 commit intomainfrom
idiomatic-branding-action
Feb 11, 2026
Merged

Simplify slide on branding in action#3089
hurryabit merged 1 commit intomainfrom
idiomatic-branding-action

Conversation

@hurryabit
Copy link
Collaborator

There's nothing to be moved into the the closures and we can hence omit the move keyord.

There's nothing to be moved into the the closures and we can hence omit
the `move` keyord.
@hurryabit hurryabit requested a review from gribozavr February 11, 2026 16:24
@hurryabit hurryabit enabled auto-merge (squash) February 11, 2026 16:24
Copy link
Collaborator

@mgeisler mgeisler left a comment

Choose a reason for hiding this comment

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

Hmm... yeah, move must have been redundant since it compiles.

This reminds me that we should be able to run Clippy on the Rust code here via dprint as in #2587.

@hurryabit hurryabit merged commit 90af359 into main Feb 11, 2026
40 checks passed
@hurryabit hurryabit deleted the idiomatic-branding-action branch February 11, 2026 20:52
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