fix(.github/workflows): reuse cached .git repository for branch HEAD commit#3893
Merged
darccio merged 8 commits intoAug 21, 2025
Conversation
…e soon-to-be merged PR as they ran on it
…flakiness is more probable while testing the PR
… test outcomes in PRs
BenchmarksBenchmark execution time: 2025-08-21 16:15:38 Comparing candidate commit f9763cc in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 24 metrics, 0 unstable metrics. |
darccio
force-pushed
the
dario.castane/ktlo/cache-checkouts-on-matrices
branch
from
August 19, 2025 16:38
4832f3a to
7b06f33
Compare
darccio
force-pushed
the
dario.castane/ktlo/cache-checkouts-on-matrices
branch
from
August 19, 2025 16:42
7b06f33 to
d39ddcb
Compare
Contributor
darccio
commented
Aug 19, 2025
Comment on lines
-4
to
-9
| workflow_call: # allows to reuse this workflow | ||
| inputs: | ||
| ref: | ||
| description: 'The branch to run the workflow on' | ||
| required: true | ||
| type: string |
Member
Author
There was a problem hiding this comment.
This is no longer needed because this workflow is never used from another one.
darccio
commented
Aug 19, 2025
| workflow_dispatch: # manually | ||
| inputs: | ||
| go-version: | ||
| description: The Go version to use |
Member
Author
There was a problem hiding this comment.
Added descriptions because a linter was complaining about them.
darccio
commented
Aug 19, 2025
Comment on lines
52
to
+56
| - name: Checkout | ||
| uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v2.7.0 | ||
| with: | ||
| ref: ${{ inputs.ref || github.ref }} | ||
| ref: ${{ github.sha }} | ||
| clean: false |
Member
Author
There was a problem hiding this comment.
If by any chance the previous step doesn't find the cached repo, checkout will just work as usual by downloading the repository. clean: false is only needed because we'll usually restore from the cached .git directory.
darccio
commented
Aug 19, 2025
| uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2 | ||
| with: | ||
| ref: ${{ inputs.ref || github.ref }} | ||
| fetch-depth: $(( ${{ github.event_name == 'pull_request' && github.event.pull_request.commits || 0 }} + 1 )) |
Member
Author
There was a problem hiding this comment.
By default fetch-depth is 1. This expression seems to end with 1 or 2, which I don't think it's required.
darccio
force-pushed
the
dario.castane/ktlo/disable-main-branch-ci
branch
from
August 21, 2025 10:05
d39ddcb to
d742705
Compare
darccio
force-pushed
the
dario.castane/ktlo/disable-main-branch-ci
branch
from
August 21, 2025 10:07
d742705 to
849e00d
Compare
…astane/ktlo/cache-checkouts-on-matrices
darccio
force-pushed
the
dario.castane/ktlo/cache-checkouts-on-matrices
branch
5 times, most recently
from
August 21, 2025 14:56
71ac868 to
4ba46f6
Compare
…th git commands to materialize from `.git`
darccio
force-pushed
the
dario.castane/ktlo/cache-checkouts-on-matrices
branch
2 times, most recently
from
August 21, 2025 15:36
7cead25 to
b8ed9e4
Compare
darccio
force-pushed
the
dario.castane/ktlo/cache-checkouts-on-matrices
branch
2 times, most recently
from
August 21, 2025 16:16
f9763cc to
4f673ed
Compare
darccio
merged commit Aug 21, 2025
8635ff7
into
dario.castane/ktlo/disable-main-branch-ci
209 of 408 checks passed
darccio
added a commit
that referenced
this pull request
Aug 22, 2025
darccio
added a commit
that referenced
this pull request
Aug 27, 2025
darccio
added a commit
that referenced
this pull request
Aug 27, 2025
darccio
added a commit
that referenced
this pull request
Aug 28, 2025
darccio
added a commit
that referenced
this pull request
Sep 3, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do?
Introduces repository caching by saving
.gitand materializing the files on checking out when running each job on the matrix.Motivation
Faster CI times even if more tests are planned to run after swapping
pull-request.ymlwithmain-branch-tests.ymlin #3887.Reviewer's Checklist
./scripts/lint.shlocally.Unsure? Have a question? Request a review!