Fix lint issues

Signed-off-by: EdricCua <[email protected]>
valkey-io · Jan 21, 2025 · 9b9de5e · 9b9de5e
1 parent 5d9a2f6
commit 9b9de5e
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 63 deletions.
diff --git a/go/api/base_client.go b/go/api/base_client.go
@@ -2953,16 +2953,15 @@ func (client *baseClient) XGroupDelConsumer(
 //
 // Return value:
 //
-//	 true if source was copied
-//		false if source was not copied.
+//	`true` if source was copied, `false` if source was not copied.
 //
 // Example:
 //
-//	 result, err := client.Copy([]string{"source, destination"})
-//		if err != nil {
-//		    // handle error
-//		}
-//		fmt.Println(result) // Output: true
+//	result, err := client.Copy([]string{"source, destination"})
+//	if err != nil {
+//	   // handle error
+//	}
+//	fmt.Println(result) // Output: true
 //
 // [valkey.io]: https://valkey.io/commands/copy/
 func (client *baseClient) Copy(source string, destination string) (bool, error) {
@@ -2987,17 +2986,16 @@ func (client *baseClient) Copy(source string, destination string) (bool, error)
 
 // Return value:
 //
-//	 true if source was copied
-//		false if source was not copied.
+//	`true` if source was copied, `false` if source was not copied.
 //
 // Example:
 //
-//	 copyOptions := api.NewCopyOptionsBuilder().SetDBDestination(2).SetReplace()
-//	 result, err := client.Copy([]string{"source, destination",copyOptions})
-//		if err != nil {
-//		    // handle error
-//		}
-//		fmt.Println(result) // Output: true
+//	copyOptions := api.NewCopyOptionsBuilder().SetDBDestination(2).SetReplace()
+//	result, err := client.Copy([]string{"source, destination",copyOptions})
+//	if err != nil {
+//	   // handle error
+//	}
+//	fmt.Println(result) // Output: true
 //
 // [valkey.io]: https://valkey.io/commands/copy/
 func (client *baseClient) CopyWithOptions(source string, destination string,

diff --git a/go/api/command_options.go b/go/api/command_options.go
@@ -388,7 +388,7 @@ func (opts *CopyOptions) toArgs() ([]string, error) {
 	if opts.replace != "" {
 		args = append(args, string(opts.replace))
 	}
-	if opts.dbDestination != 0 {
+	if opts.dbDestination != -0 {
 		args = append(args, "DB", utils.IntToString(opts.dbDestination))
 	}
 	return args, err

diff --git a/go/api/generic_base_commands.go b/go/api/generic_base_commands.go
@@ -704,54 +704,7 @@ type GenericBaseCommands interface {
 	// [valkey.io]: https://valkey.io/commands/sort/
 	SortReadOnlyWithOptions(key string, sortOptions *options.SortOptions) ([]Result[string], error)
 
-	// Copies the value stored at the source to the destination key if the
-	// destination key does not yet exist.
-	//
-	// Note:
-	//	 When in cluster mode, both source and destination must map to the same hash slot.
-	//
-	// Parameters:
-	//   source - The key to the source value.
-	//   destination - The key where the value should be copied to.
-	//
-	// Return value:
-	//  true if source was copied
-	//	false if source was not copied.
-	//
-	// Example:
-	//  result, err := client.Copy([]string{"source, destination"})
-	//	if err != nil {
-	//	    // handle error
-	//	}
-	//	fmt.Println(result) // Output: true
-	//
-	// [valkey.io]: https://valkey.io/commands/copy/
 	Copy(source string, destination string) (bool, error)
 
-	// Copies the value stored at the source to the destination key. When
-	// replace is true, removes the destination key first if it already
-	// exists, otherwise performs no action.
-	//
-	// Note:
-	//	 When in cluster mode, both source and destination must map to the same hash slot.
-	//
-	// Parameters:
-	//   source - The key to the source value.
-	//   destination - The key where the value should be copied to.
-	//   copyOptions - Set copy options with replace and DB destination-db
-
-	// Return value:
-	//  true if source was copied
-	//	false if source was not copied.
-	//
-	// Example:
-	//  copyOptions := api.NewCopyOptionsBuilder().SetDBDestination(2).SetReplace()
-	//  result, err := client.Copy([]string{"source, destination",copyOptions})
-	//	if err != nil {
-	//	    // handle error
-	//	}
-	//	fmt.Println(result) // Output: true
-	//
-	// [valkey.io]: https://valkey.io/commands/copy/
 	CopyWithOptions(source string, destination string, option *CopyOptions) (bool, error)
 }