Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix #309 Missing helptext for multiline argument #335

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Fix #309 Missing helptext for multiline argument #335

wants to merge 4 commits into from

Commits on Apr 24, 2021

  1. Fix issue google#309

    @xosnos
    xosnos committed Apr 24, 2021
    Configuration menu
    Copy the full SHA
    9496fc7 View commit details
    Browse the repository at this point in the history

  2. Extend fix for issue google#309

    @xosnos
    xosnos committed Apr 24, 2021
    Configuration menu
    Copy the full SHA
    6ad659e View commit details
    Browse the repository at this point in the history

  3. Fix style

    @xosnos
    xosnos committed Apr 24, 2021
    Configuration menu
    Copy the full SHA
    64d4a1a View commit details
    Browse the repository at this point in the history

Commits on Apr 26, 2021

  1. Make changes based on suggestions from @MichaelCG8 

    - Update comments in _consume_google_args_line to match new logic
- Fix indentation logic: indent_check compares current line indent with first line indent for argument.
- Add new function _is_arg_type to check if arg type is of valid form
- Fix tests for multiline helptexts with colons and indents
    @xosnos
    xosnos committed Apr 26, 2021
    Configuration menu
    Copy the full SHA
    434fffa View commit details
    Browse the repository at this point in the history