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

Clippy should be run in CI #287

Closed
gregtatum opened this issue Nov 10, 2022 · 2 comments
Closed

Clippy should be run in CI #287

gregtatum opened this issue Nov 10, 2022 · 2 comments
Labels
continuous-integration Issues or work for CI good first issue Want to help? Those are great bugs to start with! help wanted We need help making decisions or writing PRs for this.

Comments

@gregtatum
Copy link
Member

We should add it to a workflow in: .github/workflows
@gregtatum gregtatum added help wanted We need help making decisions or writing PRs for this. good first issue Want to help? Those are great bugs to start with! continuous-integration Issues or work for CI labels Nov 10, 2022
This was referenced Nov 18, 2022
Closed
Merged
@waywardmonkeys
Copy link
Collaborator

@gregtatum Is this something that is still desired? I have some PRs open that would be prerequisites to this like #315 (along with several other PRs).

@pitbuster pitbuster mentioned this issue Feb 14, 2024
Merged
@alerque
Copy link
Collaborator

alerque commented May 6, 2024

This was implemented in #342. I might submit some tweaks to CI usage across the board but this has been implemented.

@alerque alerque closed this as completed May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
continuous-integration Issues or work for CI good first issue Want to help? Those are great bugs to start with! help wanted We need help making decisions or writing PRs for this.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@alerque @waywardmonkeys @gregtatum