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.

Sign up for GitHub

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

Jump to bottom

Fix CI by addressing the Clippy warnings #84

Merged
merged 1 commit into from
Aug 4, 2023
Merged

Fix CI by addressing the Clippy warnings #84

merged 1 commit into from
Aug 4, 2023

Conversation

ivmarkov
Copy link
Contributor

@ivmarkov ivmarkov commented Aug 3, 2023

Subject says it all. One of the warning is really fake though.

@ivmarkov
Copy link
Contributor Author

ivmarkov commented Aug 3, 2023

By the way, are we excluding restyled from the CI? Not sure what it is doing now that we have cargo fmt --check other than failing the CI checks. :)

@kedars
Copy link
Collaborator

kedars commented Aug 4, 2023

Yes, will remove the restyled

@kedars kedars merged commit ce3bf6b into project-chip:main Aug 4, 2023
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kedars kedars kedars approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ivmarkov @kedars