Skip to content

Commit a996ba7

Browse files
committed
scripts/updatebom: remove innecessary mod override
The `GOFLAGS=-mod=mod` environment variable is not required, as that's the default mode. This code was likely copied from the tests, where `GOFLAGS` is set to `-mod=readonly`. So, executing the tool needs to override `mod`, as per the documentation, it tends to modify the `go.mod` file. Signed-off-by: Ivan Valdes <ivan@vald.es>
1 parent d52bd90 commit a996ba7

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

scripts/updatebom.sh

+1-1
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ function bom_fixlet {
1414
# shellcheck disable=SC2207
1515
modules=($(modules_for_bom))
1616

17-
if GOFLAGS=-mod=mod run_go_tool "github.com/appscodelabs/license-bill-of-materials" \
17+
if run_go_tool "github.com/appscodelabs/license-bill-of-materials" \
1818
--override-file ./bill-of-materials.override.json \
1919
"${modules[@]}" > ./bill-of-materials.json.tmp; then
2020
cp ./bill-of-materials.json.tmp ./bill-of-materials.json

0 commit comments

Comments
 (0)