Merge pull request #18603 from lekaf974/feat/rename-module-exp

Rename modules_exp to modules_for_bom to improve code and clarity
This commit is contained in:
Marek Siarkowicz 2024-09-19 10:00:30 +02:00 committed by GitHub
commit ce07474c0b
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
3 changed files with 3 additions and 3 deletions

View File

@ -509,7 +509,7 @@ function bom_pass {
log_callout "Checking bill of materials..."
# https://github.com/golang/go/commit/7c388cc89c76bc7167287fb488afcaf5a4aa12bf
# shellcheck disable=SC2207
modules=($(modules_exp))
modules=($(modules_for_bom))
# Internally license-bill-of-materials tends to modify go.sum
run cp go.sum go.sum.tmp || return 2

View File

@ -196,7 +196,7 @@ function modules() {
echo "${modules[@]}"
}
function modules_exp() {
function modules_for_bom() {
for m in $(modules); do
echo -n "${m}/... "
done

View File

@ -12,7 +12,7 @@ function bom_fixlet {
local modules
# shellcheck disable=SC2207
modules=($(modules_exp))
modules=($(modules_for_bom))
if GOFLAGS=-mod=mod run_go_tool "github.com/appscodelabs/license-bill-of-materials" \
--override-file ./bill-of-materials.override.json \