Missing support for removing group

[valllabh]

cobra/command.go

Line 1310 in b4f979a

func (c *Command) AddGroup(groups ...*Group) {

We can see there is a support to AddGroup but there is no support for RemoveGroup
Where as we also see support for AddCommand and RemoveCommand

[marckhouzam]

This seems to make sense, but due completeness can you give an example of where this would be needed?

[valllabh]

Sure @marckhouzam

Usecase:
Consider there is a Cobra based tool that is already written and maintained by the community and developer needs to use it as library to add/remove more functionalities. In such cases developer should be able to modify (add, remove) Groups just like commands.

Example:
I am facing such issue right now, where I am wrapping restish with some more functionalities.
Forking restish will lead to unnecessary maintenance.
I need to add extra functionality and make the tool more simple for the my use case .
It will also not make sense to move that code into restish.

restish is built using cobra and it has exposed root command for such use cases.
Where developer can add/remove more commands and groups. But cobra do not support removal of groups.

[JunNishimura]

@marckhouzam @valllabh

Sorry, I missed that a PR(#1912) has already been submitted regarding this Issue.
However, it appears that a PR has not been updated, so I am submitting new PR(#1997) regarding this issue just in case.

I would appreciate it if you check the PR.