-
Notifications
You must be signed in to change notification settings - Fork 511
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 for tools and run lints for pull requests #2337
Conversation
Let me run |
# For rust first-party source code | ||
if ! cargo clippy \ | ||
--manifest-path ${BUILDSYS_TOOLS_DIR}/Cargo.toml \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess the comment isn't exactly accurate yet. Are we planning to do this to first-party code in /sources
as a follow up?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I'd like first-party code to be covered as well, but it's more than I had the time to tackle today. I was working on an unrelated update to buildsys and got annoyed by the existing clippy
warnings. 😀
Signed-off-by: Ben Cressey <bcressey@amazon.com>
68f9642
to
7d3531c
Compare
Signed-off-by: Ben Cressey <bcressey@amazon.com>
Signed-off-by: Ben Cressey <bcressey@amazon.com>
Signed-off-by: Ben Cressey <bcressey@amazon.com>
Signed-off-by: Ben Cressey <bcressey@amazon.com>
7d3531c
to
a2cf343
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🦺
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
Issue number:
#1776 (partial fix)
Description of changes:
Fix or silence all clippy warnings for
tools
, and add a step to the pull request workflow so they stay fixed.Testing done:
Verified that builds completed via
buildsys
.Ran the following
pubsys
related tasks:Terms of contribution:
By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.