-
Notifications
You must be signed in to change notification settings - Fork 36
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
tests: fixes for go tests #137
Conversation
e4bea73
to
df7f087
Compare
src/test-go.bats
Outdated
} | ||
|
||
teardown_file() { | ||
for p in linux/amd64 linux/arm64 linux/ppc64le linux/s390x linux/386 linux/arm/v7 linux/arm/v6; do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some platforms are missing?
Lines 125 to 139 in 263a0bd
platforms = [ | |
"linux/386", | |
"linux/amd64", | |
"linux/arm64", | |
"linux/arm/v5", | |
"linux/arm/v6", | |
"linux/arm/v7", | |
"linux/mips", | |
"linux/mipsle", | |
"linux/mips64", | |
"linux/mips64le", | |
"linux/ppc64le", | |
"linux/s390x", | |
"linux/riscv64" | |
] |
@@ -26,9 +26,9 @@ xxadd() { | |||
|
|||
xxdel() { | |||
if [ -f /etc/alpine-release ]; then | |||
xx-apk add "$@" 2>/dev/null || true | |||
xx-apk del "$@" 2>/dev/null || true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🙈
Small lint issue: https://github.com/tonistiigi/xx/actions/runs/7419055876/job/20187947405?pr=137#step:3:267
|
Signed-off-by: Tonis Tiigi <[email protected]>
Single/double quotes have changed in the output. Signed-off-by: Tonis Tiigi <[email protected]>
Signed-off-by: Tonis Tiigi <[email protected]>
df7f087
to
2a4d47b
Compare
No description provided.