Skip to content

Commit

Permalink
internal/ci: fix reference to tool/file.RemoveAll
Browse files Browse the repository at this point in the history
After the fix for https://cuelang.org/issue/1581,
the "dry run" version of `cue cmd release` ran in CI on master
started failing as it correctly spotted a mistake:

    command.release.tidyUp: undefined field: RemoveAll:
        ./goreleaser_tool.cue:47:13

RemoveAll is declared by tool/file, not tool/os.
This was likely a mistake coming from Go's os.RemoveAll.
Thanks to the bugfix, we no longer silently skip the tidyUp step.

Signed-off-by: Daniel Martí <mvdan@mvdan.cc>
Change-Id: I5cbceb87bc40f63e299ae414ac5292478be99e50
Reviewed-on: https://review.gerrithub.io/c/cue-lang/cue/+/1177345
Unity-Result: CUE porcuepine <cue.porcuepine@gmail.com>
TryBot-Result: CUEcueckoo <cueckoo@gmail.com>
Reviewed-by: Paul Jolly <paul@myitcv.io>
  • Loading branch information
mvdan committed Feb 26, 2024
1 parent 9ef35eb commit d42ab3d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion internal/ci/goreleaser/goreleaser_tool.cue
Expand Up @@ -44,7 +44,7 @@ command: release: {

let latestCUEVersion = strings.TrimSpace(latestCUE.stdout)

tidyUp: os.RemoveAll & {
tidyUp: file.RemoveAll & {
$after: latestCUE
path: tempDir.path
}
Expand Down

0 comments on commit d42ab3d

Please sign in to comment.