Skip to content

Commit

Permalink
mr_list: remove config error message
Browse files Browse the repository at this point in the history
Any config related issues should be handled in config parsing time and not
in cmd execution. If we come to this point without detecting a wrong
user/token something else should be fixed. It's just misleading. This patch
removes it.

Signed-off-by: Bruno Meneguele <bmeneg@redhat.com>
  • Loading branch information
bmeneg committed Feb 8, 2021
1 parent 8353c1f commit e15e96c
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions cmd/mr_list.go
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ import (
"github.com/spf13/cobra"
gitlab "github.com/xanzy/go-gitlab"
"github.com/zaquestion/lab/internal/action"
"github.com/zaquestion/lab/internal/config"
lab "github.com/zaquestion/lab/internal/gitlab"
)

Expand Down Expand Up @@ -47,8 +46,7 @@ lab mr list remote "search terms" # search "remote" for merge requests with "se
Run: func(cmd *cobra.Command, args []string) {
mrs, err := mrList(args)
if err != nil {
log.Print(err)
config.UserConfigError()
log.Fatal(err)
}

pager := NewPager(cmd.Flags())
Expand Down

0 comments on commit e15e96c

Please sign in to comment.