Commit abc10f0
committed
ci: checkout first, disable go caching
First, checking out the code should be done before installing Go.
Second, since we don't have top-level go.mod, actions/setup-go
complains:
> Restore cache failed: Dependencies file is not found in /home/runner/work/sys/sys. Supported file pattern: go.sum
One way to solve this would be to
cache-dependency-path: "*/go.sum"
parameter to actions/setup-go. Alas it won't work because not all
modules here have go.mod, and * in GHA is not what you think it is
(i.e. not a part of shell glob pattern, but rather "every directory"),
and as a result it complains about missing go.sum files.
Another way is to list all the paths explicitly, which is not good from
the maintainability perspective (someone will definitely forgot to add
a go.sum path).
Yet another way is to add a step which lists all go.sum. It works
something like this:
- name: Find go.sum files
id: gosum
run: |
echo 'files<<EOF' >> "$GITHUB_OUTPUT"
git ls-files '*/go.sum' >> "$GITHUB_OUTPUT"
echo 'EOF' >> "$GITHUB_OUTPUT"
...
- name: Install Go
uses: actions/setup-go@v5
with:
cache-dependency-path: ${{ steps.gosum.outputs.files }}
But given the fact that caching is not doing much here (as we don't have
a lot of code) it's becoming too complicated with not much to gain.
So, let's just disable Go caching, so the warning above will go away.
Signed-off-by: Kir Kolyshkin <[email protected]>1 parent 2a3520a commit abc10f0
1 file changed
+6
-2
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| 14 | + | |
| 15 | + | |
14 | 16 | | |
15 | 17 | | |
16 | 18 | | |
17 | 19 | | |
18 | | - | |
19 | | - | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
20 | 24 | | |
21 | 25 | | |
22 | 26 | | |
| |||
0 commit comments