Skip to content

Improve efficiency of CI workflow through several small changes#1042

Merged
mhucka merged 6 commits intoquantumlib:mainfrom
mhucka:minor-workflow-updates
Apr 9, 2026
Merged

Improve efficiency of CI workflow through several small changes#1042
mhucka merged 6 commits intoquantumlib:mainfrom
mhucka:minor-workflow-updates

Conversation

@mhucka
Copy link
Copy Markdown
Collaborator

@mhucka mhucka commented Apr 8, 2026

Efficiency changes:

  • Most of the CI jobs don't need the full git history. We can remove fetch-depth: 0 when calling actions/checkout in those jobs.

  • Remove find-xargs pipelines when calling linters. It adds complexity and is unlikely to result in performance improvements for small repos like most of ours.

  • Remove Scorecard data upload for debugging, because we never used it.

  • Run the pr-labeler workflow on ubuntu-slim runners.

mhucka added 6 commits April 8, 2026 04:23
Remove `depth: 0` from `actions/checkout` jobs that don't need it.
There's little to be gained in efficiency by prefiltering the files
passed to yamllint. Skip running `find ... | xargs`.
There's little to be gained in efficiency by prefiltering the files
passed to buildifier. Skip running `find ... | xargs`.
Finding the Dockerfiles can be done more efficiently than running a find
and xargs pipeline.
@github-actions github-actions bot added the size: S 10< lines changed <50 label Apr 8, 2026
@mhucka mhucka marked this pull request as ready for review April 8, 2026 06:42
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@mhucka mhucka requested a review from pavoljuhas April 8, 2026 06:42
@mhucka mhucka added this pull request to the merge queue Apr 9, 2026
Merged via the queue into quantumlib:main with commit 2533066 Apr 9, 2026
31 checks passed
@mhucka mhucka deleted the minor-workflow-updates branch April 9, 2026 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: S 10< lines changed <50

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants