MichaelCG8
MichaelCG8
Nice one! Got a couple of thoughts. - There's a pre-existing comment in `_consumer_google_args_line()` that says `# first is either the "arg" or "arg (type)"`. Could you update that now...
Looks great! I think this can be merged. One minor thing that you may or may not wish to address is that your new function `_is_arg_type()` will actually return `None`...
Hi, I'm exploring fire for the first time and up for having a go at this one. Could you point me towards the best place in the code to start...
Okay, so it looks like the target section is `_ArgsAndFlagsSection()`, which calls `_CreateFlagItem()`, both in helptext.py At the moment `_CreateFlagItem()` takes a `docstring_info` argument to include the docstring description of...
Hi, I have a solution ready and would like to create a pull request. However, when I try to push my branch I get the error message: remote: Permission to...
Great, thanks. I've put in pull request #251.
`fire.docstrings._consume_google_args_line()` processes the missing line as though it is an arg and type. Potentially fixable by being more selective about what constitutes a new arg line (not just that it...
Thanks @xosnos ! If you'd like me to review anything let me know :)
Change added in pull request #251
I've just ran pylint and it looks like I got some of the indentation wrong, so I'll fix that tomorrow. Also running pylint with a py2 interpreter doesn't like some...