Skip to content

Commit

Permalink
issue,mr: remove markdown flag from parent command
Browse files Browse the repository at this point in the history
The flag "--no-markdown","-M" was being shown in both 'lab mr' and 'lab issue'
command as a possible flag, but it had no effect whatsoever in any commands
other than in the 'show' subcommands, which had their own implementation of
this very same flag. With that, remove it from the the parent commands to avoid
confusion on its usage.

Signed-off-by: Bruno Meneguele <bmeneg@redhat.com>
  • Loading branch information
bmeneg committed Sep 8, 2020
1 parent 146904c commit 08f5c32
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 2 deletions.
1 change: 0 additions & 1 deletion cmd/issue.go
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,5 @@ func init() {
issueCmd.Flags().BoolP("list", "l", false, "List issues on a remote")
issueCmd.Flags().BoolP("browse", "b", false, "View issue <id> in a browser")
issueCmd.Flags().StringP("close", "d", "", "Close issue <id> on remote")
issueCmd.Flags().BoolP("no-markdown", "M", false, "Don't use markdown renderer to print the issue description")
RootCmd.AddCommand(issueCmd)
}
1 change: 0 additions & 1 deletion cmd/mr.go
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,5 @@ func init() {
mrCmd.Flags().BoolP("list", "l", false, "List merge requests on a remote")
mrCmd.Flags().BoolP("browse", "b", false, "View merge request <id> in a browser")
mrCmd.Flags().StringP("close", "d", "", "Close merge request <id> on remote")
mrCmd.Flags().BoolP("no-markdown", "M", false, "Don't use markdown renderer to print the issue description")
RootCmd.AddCommand(mrCmd)
}

0 comments on commit 08f5c32

Please sign in to comment.