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

[CI] refactor clippy tests and fix clippy CI OOM #1470

Merged
merged 10 commits into from
Oct 13, 2023
Merged

[CI] refactor clippy tests and fix clippy CI OOM #1470

merged 10 commits into from
Oct 13, 2023

Conversation

clangenb
Copy link
Contributor

@clangenb clangenb commented Oct 13, 2023 •
edited
Loading

Fixing it by putting the individual tests into a matrix that runs them in parallel on different machines. This also reduces the total time to run those clippy tests (less than linearly though, as the subsequent tests partly benefited from the cargo cache of the test before).

@clangenb clangenb added B1-releasenotes C1-low 📌 Does not elevate a release containing this beyond "low priority" E0-breaksnothing A7-somethingelse labels Oct 13, 2023
@clangenb clangenb requested review from brenzi and mosonyi October 13, 2023 06:56
@clangenb clangenb merged commit 28baa5b into master Oct 13, 2023
32 checks passed
@clangenb clangenb deleted the cl/ci-refactor-clippy-tests branch October 13, 2023 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@brenzi brenzi Awaiting requested review from brenzi

@mosonyi mosonyi Awaiting requested review from mosonyi

Assignees
No one assigned
Labels
A7-somethingelse B1-releasenotes C1-low 📌 Does not elevate a release containing this beyond "low priority" E0-breaksnothing
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

clippy runs out of memory
1 participant
@clangenb