starred/cli
1
0
Fork 0
mirror of https://github.com/docker/cli.git synced 2026-08-28 10:05:17 -05:00
Code Issues Packages Projects Releases Wiki Activity
11,614 Commits 13 Branches 320 Tags
v28.5.2
Commit Graph
3 Commits
Author SHA1 Message Date
Sebastiaan van Stijn 762b5a8df3 opts: remove redundant capturing of loop vars in tests (copyloopvar) 
go1.22 and up now produce a unique variable in loops, tehrefore no longer
requiring to capture the variable manually;

    service/logs/parse_logs_test.go:50:3: The copy of the 'for' variable "tc" can be deleted (Go 1.22+) (copyloopvar)
            tc := tc
            ^

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2024-11-05 10:14:31 +01:00
Sebastiaan van Stijn 28b0aa9f1a replace uses of deprecated env.Patch() 
Also removing redundant defer for env.PatchAll(), which is now automatically
handled in t.Cleanup()

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2022-09-22 17:28:07 +02:00
Sebastiaan van Stijn 973713bc05 ReadKVEnvStrings/ReadKVStrings return nil if empty, and add tests 
If no env-vars were loaded from "files", and "overrides" was nil,
the code returned an empty slice instead of a `nil` value.

Also add a test for this function, as no unit test was present yet.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2020-06-26 16:07:13 +02:00