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

Fix clippy warnings in the codebase #477

Closed
mstallmo opened this issue Dec 29, 2018 · 1 comment · Fixed by #478
Closed

Fix clippy warnings in the codebase #477

mstallmo opened this issue Dec 29, 2018 · 1 comment · Fixed by #478
Assignees
Labels
Milestone

Comments

@mstallmo
Copy link
Member

While working on a different refactor I ran clippy on the codebase and noticed that it gave a handful of warnings about various parts of the code. We should probably refactor the flagged code to be in line with the clippy lints.

Along this line are there any clippy lints that we want to ignore/diverge from or should we follow all of the linting rules?

@ashleygwilliams ashleygwilliams added PR attached there's a PR open for this issue to-do stuff that needs to happen, so plz do it k thx refactor labels Jan 2, 2019
@ashleygwilliams
Copy link
Member

hey @mstallmo thanks for tackling this! in general, i'm open to the clippy defaults, my primary concern is balancing the clippy lints with code readability (for maintainability sake). i'll take a look at your PR and comment there!

@ashleygwilliams ashleygwilliams added changelog - maintenance and removed to-do stuff that needs to happen, so plz do it k thx labels Jan 15, 2019
@ashleygwilliams ashleygwilliams modified the milestones: 0.6.0, 0.7.0 Jan 15, 2019
@ashleygwilliams ashleygwilliams modified the milestones: 0.8.0, 0.7.0 Mar 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants