fix/added fixes recommended in copilot review

Signed-off-by: Rayyan Khan <rayyanrehman101@gmail.com>

Author: rkcoder101

cmd/harbor/root/project/list.go

@@ -56,7 +56,7 @@ func BuildListOptions(private, public bool, opts *api.ListFlags, fuzzy, match, r
 	}
 
 	if private && public {
-		return nil, fmt.Errorf("Cannot specify both --private and --public flags")
+		return nil, fmt.Errorf("cannot specify both --private and --public flags")
 	}
 
 	if private {
@@ -107,7 +107,7 @@ func ListProjectCommand() *cobra.Command {
 			log.Debug("Fetching projects...")
 			allProjects, err = fetchProjects(listFunc, opts)
 			if err != nil {
-				return fmt.Errorf("%s", utils.ParseHarborErrorMsg(err))
+				return fmt.Errorf("failed to get projects list: %s", utils.ParseHarborErrorMsg(err))
 			}
 			if err := PrintProjects(allProjects); err != nil {
 				return err

cmd/harbor/root/project/list_test.go

@@ -31,7 +31,7 @@ import (
 	log "github.com/sirupsen/logrus"
 	"github.com/spf13/viper"
 	"github.com/stretchr/testify/assert"
-	"go.yaml.in/yaml/v4"
+	"gopkg.in/yaml.v2"
 )
 
 func captureOutput(f func() error) (string, error) {
@@ -71,7 +71,7 @@ func (m *MockProjectLister) mockListFunc(opts ...api.ListFlags) (project.ListPro
 		return *res, fmt.Errorf("mock list error")
 	}
 	if len(opts) == 0 {
-		return *res, fmt.Errorf("No options passed")
+		return *res, fmt.Errorf("no options passed")
 	}
 	listFlags := opts[0]
 	page, pageSize := listFlags.Page, listFlags.PageSize
@@ -158,7 +158,7 @@ func TestBuildListOptions(t *testing.T) {
 					opts:    &api.ListFlags{},
 				}
 			},
-			wantedErr: "Cannot specify both --private and --public",
+			wantedErr: "cannot specify both --private and --public",
 		},
 		{
 			name: "page size exceeds maximum",

pkg/views/project/list/view.go

@@ -61,7 +61,7 @@ func ListProjects(projects []*models.Project) error {
 	m := tablelist.NewModel(columns, rows, len(rows))
 
 	if _, err := tea.NewProgram(m, tea.WithOutput(os.Stdout), tea.WithInput(nil)).Run(); err != nil {
-		return fmt.Errorf("Error running program:%w", err)
+		return fmt.Errorf("Error running program: %w", err)
 	}
 	return nil
 }