Skip to content

Commit 4c90ff9

Browse files
refactor: add comments explaining Docker incompatibility reason
Signed-off-by: Ogulcan Aydogan <ogulcanaydogan@hotmail.com>
1 parent 5ee828b commit 4c90ff9

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

cmd/nerdctl/container/multi_platform_linux_test.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -88,6 +88,7 @@ func TestMultiPlatformBuildPush(t *testing.T) {
8888
testCase := nerdtest.Setup()
8989

9090
testCase.Require = require.All(
91+
// non-buildx `docker build` lacks multi-platform support; `docker push` lacks --platform
9192
require.Not(nerdtest.Docker),
9293
nerdtest.Build,
9394
requireMultiPlatformExec,
@@ -134,6 +135,7 @@ func TestMultiPlatformBuildPushNoRun(t *testing.T) {
134135
testCase := nerdtest.Setup()
135136

136137
testCase.Require = require.All(
138+
// non-buildx `docker build` lacks multi-platform support; `docker push` lacks --platform
137139
require.Not(nerdtest.Docker),
138140
nerdtest.Build,
139141
requireMultiPlatformExec,

0 commit comments

Comments
 (0)