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

chore: address new clippy 0.1.51 warnings #364

Merged
merged 6 commits into from
Mar 25, 2021

Conversation

EverlastingBugstopper
Copy link
Contributor

@EverlastingBugstopper EverlastingBugstopper commented Mar 25, 2021

new clippy update enforces rusty naming conventions and disallows capitalizing acronyms in enum variants/struct names. we had to ignore those clippy warnings in our rover-client/query module because all of that stuff is autogenerated and there's nothing we can do.

also wanted us to implement From instead of Into for GitContext since From gives you Into for free while that is not the case the other way around.

none of this changes any behavior.

@EverlastingBugstopper EverlastingBugstopper changed the title chore: s/IOError/IoError chore: address new clippy 0.1.51 warnings Mar 25, 2021
@EverlastingBugstopper EverlastingBugstopper self-assigned this Mar 25, 2021
@EverlastingBugstopper EverlastingBugstopper merged commit fc7fe73 into main Mar 25, 2021
@EverlastingBugstopper EverlastingBugstopper deleted the avery/fix-clippy branch March 25, 2021 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants