Skip to content

Comments

Few fixes and enhancements for the CLI#22

Open
ThomasFaivre wants to merge 10 commits intorawrgulmuffins:0.6.0.devfrom
ThomasFaivre:0.6.0.dev
Open

Few fixes and enhancements for the CLI#22
ThomasFaivre wants to merge 10 commits intorawrgulmuffins:0.6.0.devfrom
ThomasFaivre:0.6.0.dev

Conversation

@ThomasFaivre
Copy link

Hello!

I have started using bugzillatools commands recently and there were some stuff that I thought was missing for the cli. So I added some stuff and fixed some bugs on the way.

What do you think?

PS: I am not very familiar with the GitHub process, so if you would like to split the enhancement and the fixes, just tell me ;-)

Signed-off-by: Thomas Faivre <thomas.faivre@6wind.com>
Signed-off-by: Thomas Faivre <thomas.faivre@6wind.com>
Fixes the following issue:
$ bugzilla help
usage: bugzilla [-h] [-V] {} ...
bugzilla: error: invalid choice: 'help' (choose from )

Fixes: 39e0465 ("Down to just one attribute error.")
Signed-off-by: Thomas Faivre <thomas.faivre@6wind.com>
Signed-off-by: Thomas Faivre <thomas.faivre@6wind.com>
Signed-off-by: Thomas Faivre <thomas.faivre@6wind.com>
This command can dump or set any field of a bug

Signed-off-by: Thomas Faivre <thomas.faivre@6wind.com>
This decorator allows to add a group of arguments.

The with_server decorator has been replaced by this decorator.
Search criteria for the 'search' command has been added to a separate
group to improve clarity

Signed-off-by: Thomas Faivre <thomas.faivre@6wind.com>
Only display the numbers of the matched bugs and not the summary.

Signed-off-by: Thomas Faivre <thomas.faivre@6wind.com>
Signed-off-by: Thomas Faivre <thomas.faivre@6wind.com>
Signed-off-by: Thomas Faivre <thomas.faivre@6wind.com>
@brookskindle brookskindle requested review from rawrgulmuffins and removed request for rawrgulmuffins April 14, 2017 17:27
@rawrgulmuffins
Copy link
Owner

rawrgulmuffins commented Apr 18, 2017

Hello and thank you for the PR. Brooks and I are both working on what our process should look like on our end. Sorry for the delay and we'd love to get feedback on changes you think should be implemented at a project level.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants