fixed go lint warnings

Author: cmaglie

Diff for: commands/errors.go

@@ -32,6 +32,7 @@ func composeErrorMsg(msg string, cause error) string {
 
 // CommandError is an error that may be converted into a gRPC status.
 type CommandError interface {
+	// ToRPCStatus convertes the error into a *status.Status
 	ToRPCStatus() *status.Status
 }
 
@@ -42,6 +43,7 @@ func (e *InvalidInstanceError) Error() string {
 	return tr("Invalid instance")
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *InvalidInstanceError) ToRPCStatus() *status.Status {
 	return status.New(codes.InvalidArgument, e.Error())
 }
@@ -55,6 +57,7 @@ func (e *InvalidFQBNError) Error() string {
 	return composeErrorMsg(tr("Invalid FQBN"), e.Cause)
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *InvalidFQBNError) ToRPCStatus() *status.Status {
 	return status.New(codes.InvalidArgument, e.Error())
 }
@@ -72,6 +75,7 @@ func (e *InvalidURLError) Error() string {
 	return composeErrorMsg(tr("Invalid URL"), e.Cause)
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *InvalidURLError) ToRPCStatus() *status.Status {
 	return status.New(codes.InvalidArgument, e.Error())
 }
@@ -89,6 +93,7 @@ func (e *InvalidLibraryError) Error() string {
 	return composeErrorMsg(tr("Invalid library"), e.Cause)
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *InvalidLibraryError) ToRPCStatus() *status.Status {
 	return status.New(codes.InvalidArgument, e.Error())
 }
@@ -106,6 +111,7 @@ func (e *InvalidVersionError) Error() string {
 	return composeErrorMsg(tr("Invalid version"), e.Cause)
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *InvalidVersionError) ToRPCStatus() *status.Status {
 	return status.New(codes.InvalidArgument, e.Error())
 }
@@ -121,6 +127,7 @@ func (e *MissingFQBNError) Error() string {
 	return tr("Missing FQBN (Fully Qualified Board Name)")
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *MissingFQBNError) ToRPCStatus() *status.Status {
 	return status.New(codes.InvalidArgument, e.Error())
 }
@@ -138,6 +145,7 @@ func (e *UnknownFQBNError) Unwrap() error {
 	return e.Cause
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *UnknownFQBNError) ToRPCStatus() *status.Status {
 	return status.New(codes.NotFound, e.Error())
 }
@@ -149,6 +157,7 @@ func (e *MissingPortProtocolError) Error() string {
 	return tr("Missing port protocol")
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *MissingPortProtocolError) ToRPCStatus() *status.Status {
 	return status.New(codes.InvalidArgument, e.Error())
 }
@@ -160,6 +169,7 @@ func (e *MissingProgrammerError) Error() string {
 	return tr("Missing programmer")
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *MissingProgrammerError) ToRPCStatus() *status.Status {
 	return status.New(codes.InvalidArgument, e.Error())
 }
@@ -171,6 +181,7 @@ func (e *ProgreammerRequiredForUploadError) Error() string {
 	return tr("A programmer is required to upload")
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *ProgreammerRequiredForUploadError) ToRPCStatus() *status.Status {
 	st, _ := status.
 		New(codes.InvalidArgument, e.Error()).
@@ -192,6 +203,7 @@ func (e *ProgrammerNotFoundError) Unwrap() error {
 	return e.Cause
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *ProgrammerNotFoundError) ToRPCStatus() *status.Status {
 	return status.New(codes.NotFound, e.Error())
 }
@@ -206,6 +218,7 @@ func (e *InvalidPlatformPropertyError) Error() string {
 	return tr("Invalid '%[1]s' property: %[2]s", e.Property, e.Value)
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *InvalidPlatformPropertyError) ToRPCStatus() *status.Status {
 	return status.New(codes.FailedPrecondition, e.Error())
 }
@@ -219,6 +232,7 @@ func (e *MissingPlatformPropertyError) Error() string {
 	return tr("Property '%s' is undefined", e.Property)
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *MissingPlatformPropertyError) ToRPCStatus() *status.Status {
 	return status.New(codes.FailedPrecondition, e.Error())
 }
@@ -233,6 +247,7 @@ func (e *PlatformNotFound) Error() string {
 	return composeErrorMsg(tr("Platform '%s' not found", e.Platform), e.Cause)
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *PlatformNotFound) ToRPCStatus() *status.Status {
 	return status.New(codes.FailedPrecondition, e.Error())
 }
@@ -251,6 +266,7 @@ func (e *LibraryNotFound) Error() string {
 	return composeErrorMsg(tr("Library '%s' not found", e.Library), e.Cause)
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *LibraryNotFound) ToRPCStatus() *status.Status {
 	return status.New(codes.FailedPrecondition, e.Error())
 }
@@ -269,6 +285,7 @@ func (e *LibraryDependenciesResolutionFailedError) Error() string {
 	return composeErrorMsg(tr("No valid dependencies solution found"), e.Cause)
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *LibraryDependenciesResolutionFailedError) ToRPCStatus() *status.Status {
 	return status.New(codes.FailedPrecondition, e.Error())
 }
@@ -286,6 +303,7 @@ func (e *PlatformAlreadyAtTheLatestVersionError) Error() string {
 	return tr("Platform '%s' is already at the latest version", e.Platform)
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *PlatformAlreadyAtTheLatestVersionError) ToRPCStatus() *status.Status {
 	st, _ := status.
 		New(codes.AlreadyExists, e.Error()).
@@ -300,6 +318,7 @@ func (e *MissingSketchPathError) Error() string {
 	return tr("Missing sketch path")
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *MissingSketchPathError) ToRPCStatus() *status.Status {
 	return status.New(codes.InvalidArgument, e.Error())
 }
@@ -317,6 +336,7 @@ func (e *CantOpenSketchError) Unwrap() error {
 	return e.Cause
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *CantOpenSketchError) ToRPCStatus() *status.Status {
 	return status.New(codes.NotFound, e.Error())
 }
@@ -335,6 +355,7 @@ func (e *FailedInstallError) Unwrap() error {
 	return e.Cause
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *FailedInstallError) ToRPCStatus() *status.Status {
 	return status.New(codes.Internal, e.Error())
 }
@@ -352,6 +373,7 @@ func (e *FailedLibraryInstallError) Unwrap() error {
 	return e.Cause
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *FailedLibraryInstallError) ToRPCStatus() *status.Status {
 	return status.New(codes.Internal, e.Error())
 }
@@ -370,6 +392,7 @@ func (e *FailedUninstallError) Unwrap() error {
 	return e.Cause
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *FailedUninstallError) ToRPCStatus() *status.Status {
 	return status.New(codes.Internal, e.Error())
 }
@@ -388,6 +411,7 @@ func (e *FailedDownloadError) Unwrap() error {
 	return e.Cause
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *FailedDownloadError) ToRPCStatus() *status.Status {
 	return status.New(codes.Internal, e.Error())
 }
@@ -406,6 +430,7 @@ func (e *FailedUploadError) Unwrap() error {
 	return e.Cause
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *FailedUploadError) ToRPCStatus() *status.Status {
 	return status.New(codes.Internal, e.Error())
 }
@@ -424,6 +449,7 @@ func (e *FailedDebugError) Unwrap() error {
 	return e.Cause
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *FailedDebugError) ToRPCStatus() *status.Status {
 	return status.New(codes.Internal, e.Error())
 }
@@ -442,6 +468,7 @@ func (e *CompileFailedError) Unwrap() error {
 	return e.Cause
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *CompileFailedError) ToRPCStatus() *status.Status {
 	return status.New(codes.Internal, e.Error())
 }
@@ -460,6 +487,7 @@ func (e *InvalidArgumentError) Unwrap() error {
 	return e.Cause
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *InvalidArgumentError) ToRPCStatus() *status.Status {
 	return status.New(codes.InvalidArgument, e.Error())
 }
@@ -478,6 +506,7 @@ func (e *NotFoundError) Unwrap() error {
 	return e.Cause
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *NotFoundError) ToRPCStatus() *status.Status {
 	return status.New(codes.NotFound, e.Error())
 }
@@ -496,6 +525,7 @@ func (e *PermissionDeniedError) Unwrap() error {
 	return e.Cause
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *PermissionDeniedError) ToRPCStatus() *status.Status {
 	return status.New(codes.PermissionDenied, e.Error())
 }
@@ -514,6 +544,7 @@ func (e *UnavailableError) Unwrap() error {
 	return e.Cause
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *UnavailableError) ToRPCStatus() *status.Status {
 	return status.New(codes.Unavailable, e.Error())
 }
@@ -531,6 +562,7 @@ func (e *TempDirCreationFailedError) Unwrap() error {
 	return e.Cause
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *TempDirCreationFailedError) ToRPCStatus() *status.Status {
 	return status.New(codes.Unavailable, e.Error())
 }
@@ -548,6 +580,7 @@ func (e *TempFileCreationFailedError) Unwrap() error {
 	return e.Cause
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *TempFileCreationFailedError) ToRPCStatus() *status.Status {
 	return status.New(codes.Unavailable, e.Error())
 }
@@ -566,6 +599,7 @@ func (e *SignatureVerificationFailedError) Unwrap() error {
 	return e.Cause
 }
 
+// ToRPCStatus convertes the error into a *status.Status
 func (e *SignatureVerificationFailedError) ToRPCStatus() *status.Status {
 	return status.New(codes.Unavailable, e.Error())
 }