Skip to content

Conversation

@Cierra-Runis
Copy link
Contributor

@Cierra-Runis Cierra-Runis commented May 9, 2025

🛠️ Refactor Code with Clippy (No Logic Change)

This pull request refactors the codebase by applying suggestions from clippy, Rust’s built-in linter. The changes aim to improve code readability, consistency, and style without introducing any functional or behavioral modifications.

✅ Summary of Changes:

  • Simplified nested if let and match statements (e.g., collapsible matches)
  • Removed redundant clones and bindings
  • Applied idiomatic Rust patterns where safe and clear
  • Ensured all changes pass existing tests and preserve original logic

This is a non-functional cleanup PR, safe for review and merge without risk of regression.

@AlexCharlton
Copy link
Owner

Thanks for the contribution, @Cierra-Runis!

Would you mind throwing a license file in here while you're at it, as you suggested?

I'd also suggest updating the CI workflow to include a clippy check, to ensure that this standard will be maintained.

@Cierra-Runis
Copy link
Contributor Author

I'd love to❤️. However, it's best if each pull request addresses only one issue or feature for clarity and easier review. Next Pull Request is coming😘.

@AlexCharlton AlexCharlton merged commit b49afc2 into AlexCharlton:master May 12, 2025
1 check passed
@AlexCharlton
Copy link
Owner

Great, thanks!

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