Apply suggestions from code review

Co-authored-by: per1234 <[email protected]>

Author: cmaglie

commands/instances.go

@@ -75,7 +75,7 @@ func (c *coreInstancesContainer) GetInstance(id int32) *CoreInstance {
 	return c.instances[id]
 }
 
-// AddAndAssignID saves the CoreInstance and assign an unique ID to
+// AddAndAssignID saves the CoreInstance and assigns a unique ID to
 // retrieve it later
 func (c *coreInstancesContainer) AddAndAssignID(i *CoreInstance) int32 {
 	c.instancesMux.Lock()
@@ -86,9 +86,9 @@ func (c *coreInstancesContainer) AddAndAssignID(i *CoreInstance) int32 {
 	return id
 }
 
-// RemoveID removed the CoreInstance referenced by id. Returns true
+// RemoveID removes the CoreInstance referenced by id. Returns true
 // if the operation is successful, or false if the CoreInstance does
-// not exists
+// not exist
 func (c *coreInstancesContainer) RemoveID(id int32) bool {
 	c.instancesMux.Lock()
 	defer c.instancesMux.Unlock()

docs/UPGRADING.md

@@ -232,7 +232,7 @@ func LibraryUpgrade(instanceID int32, libraryNames []string, downloadCB rpc.Down
 have been changed to:
 
 ```go
-func Watch(req *rpc.BoardListWatchRequest) (<-chan *rpc.BoardListWatchResponse, func(), error) { ... ]
+func Watch(req *rpc.BoardListWatchRequest) (<-chan *rpc.BoardListWatchResponse, func(), error) { ... }
 func LibraryUpgradeAll(req *rpc.LibraryUpgradeAllRequest, downloadCB rpc.DownloadProgressCB, taskCB rpc.TaskProgressCB) error { ... }
 func LibraryUpgrade(ctx context.Context, req *rpc.LibraryUpgradeRequest, downloadCB rpc.DownloadProgressCB, taskCB rpc.TaskProgressCB) error { ... }
 ```
@@ -249,11 +249,11 @@ have been changed to:
 
 ```go
 func GetPackageManager(instance rpc.InstanceCommand) *packagemanager.PackageManager { ... } // Deprecated
-func GetPackageManagerExplorer(req rpc.InstanceCommand) (explorer *packagemanager.Explorer, release func()) { ... ]
+func GetPackageManagerExplorer(req rpc.InstanceCommand) (explorer *packagemanager.Explorer, release func()) { ... }
 func GetLibraryManager(req rpc.InstanceCommand) *librariesmanager.LibrariesManager { ... }
 ```
 
-Old code using passing the `instanceID` inside the gRPC request must be changed to pass directly the whole gRPC request,
+Old code passing the `instanceID` inside the gRPC request must be changed to pass directly the whole gRPC request,
 for example:
 
 ```go

rpc/cc/arduino/cli/commands/v1/common.go

@@ -21,7 +21,7 @@ type DownloadProgressCB func(curr *DownloadProgress)
 // TaskProgressCB is a callback to receive progress messages
 type TaskProgressCB func(msg *TaskProgress)
 
-// InstanceCommand is an interface thar represents a gRPC command with
+// InstanceCommand is an interface that represents a gRPC command with
 // a gRPC Instance.
 type InstanceCommand interface {
 	GetInstance() *Instance