Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use Clippy tool optimized code #171

Merged
merged 2 commits into from
Mar 11, 2022
Merged

Use Clippy tool optimized code #171

merged 2 commits into from
Mar 11, 2022

Conversation

mrxiaozhuox
Copy link
Contributor

Description

Use Clippy tool optimized code.

@netlify
Copy link

netlify bot commented Mar 9, 2022

✔️ Deploy Preview for robyn canceled.

🔨 Explore the source changes: c2a99a9

🔍 Inspect the deploy log: https://app.netlify.com/sites/robyn/deploys/62280a6ce56dfd000758698e

@sansyrox
Copy link
Member

Hi @mrxiaozhuox . Thank you for your contribution. This looks good from first glance.

@sansyrox sansyrox merged commit edd51d0 into sparckles:main Mar 11, 2022
@sansyrox
Copy link
Member

Hey @mrxiaozhuox ,

What were the steps used by you for this optimization?

@mrxiaozhuox
Copy link
Contributor Author

What were the steps used by you for this optimization?

I've change my ra use clippy check, so clippy can give you some optimization scheme.

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