network: WithCheckDuplicate: don't set CheckDuplicate value (#2589)

Setting this value is a no-opt, as it's automatically set by the client,
and the field will be removed in the next release of the docker module,
so let's already remove it.

updates 6e086e8, which deprecated this
function.

Signed-off-by: Sebastiaan van Stijn <[email protected]>

Author: thaJeztah

docker_examples_test.go

@@ -196,7 +196,6 @@ func ExampleNew_withNetwork() {
 	ctx := context.Background()
 
 	net, err := NewNetwork(ctx,
-		network.WithCheckDuplicate(),
 		network.WithAttachable(),
 		// Makes the network internal only, meaning the host machine cannot access it.
 		// Remove or use `network.WithDriver("bridge")` to change the network's mode.

network/options.go

@@ -32,9 +32,6 @@ func WithAttachable() CustomizeOption {
 // Deprecated: CheckDuplicate is deprecated since API v1.44, but it defaults to true when sent by the client package to older daemons.
 func WithCheckDuplicate() CustomizeOption {
 	return func(original *types.NetworkCreate) error {
-		//nolint:staticcheck
-		original.CheckDuplicate = true
-
 		return nil
 	}
 }