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.

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

Fix incorrect markdown formatting #671

Closed
wants to merge 8 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ dist
.idea
site
.coverage
.coverage.*
htmlcov
.pytest_cache
coverage.xml
Expand Down
18 changes: 18 additions & 0 deletions tests/test_others.py
Original file line number Diff line number Diff line change
Expand Up @@ -256,3 +256,21 @@ def test_split_opt():
prefix, opt = _split_opt("verbose")
assert prefix == ""
assert opt == "verbose"


def test_options_metadata_typer_default():
app = typer.Typer(options_metavar="[options]")

@app.command()
def c1():
pass # pragma: nocover

@app.command(options_metavar="[OPTS]")
def c2():
pass # pragma: nocover

result = runner.invoke(app, ["c1", "--help"])
assert "Usage: root c1 [options]" in result.stdout

result = runner.invoke(app, ["c2", "--help"])
assert "Usage: root c2 [OPTS]" in result.stdout
26 changes: 26 additions & 0 deletions tests/test_rich_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -50,3 +50,29 @@ def main() -> None:

assert result.exit_code == 0
assert "Show this message" in result.stdout


def test_markdown_pars():
app = typer.Typer(rich_markup_mode="markdown")

@app.command()
def main():
"""First line

Line 1

Line 2
"""

result = runner.invoke(app, ["--help"])
lines = [line.strip() for line in result.stdout.split("\n")]
help_start = lines.index("First line")
assert help_start != -1
assert lines[help_start : help_start + 6] == [
"First line",
"",
"Line 1",
"",
"Line 2",
"",
]
20 changes: 14 additions & 6 deletions typer/main.py
Original file line number Diff line number Diff line change
Expand Up @@ -181,7 +181,7 @@ def callback(
help: Optional[str] = Default(None),
epilog: Optional[str] = Default(None),
short_help: Optional[str] = Default(None),
options_metavar: str = Default("[OPTIONS]"),
options_metavar: Optional[str] = None,
add_help_option: bool = Default(True),
hidden: bool = Default(False),
deprecated: bool = Default(False),
Expand All @@ -202,7 +202,7 @@ def decorator(f: CommandFunctionType) -> CommandFunctionType:
help=help,
epilog=epilog,
short_help=short_help,
options_metavar=options_metavar,
options_metavar=options_metavar or self.info.options_metavar,
add_help_option=add_help_option,
hidden=hidden,
deprecated=deprecated,
Expand All @@ -221,7 +221,7 @@ def command(
help: Optional[str] = None,
epilog: Optional[str] = None,
short_help: Optional[str] = None,
options_metavar: str = "[OPTIONS]",
options_metavar: Optional[str] = None,
add_help_option: bool = True,
no_args_is_help: bool = False,
hidden: bool = False,
Expand All @@ -242,7 +242,9 @@ def decorator(f: CommandFunctionType) -> CommandFunctionType:
help=help,
epilog=epilog,
short_help=short_help,
options_metavar=options_metavar,
options_metavar=(
options_metavar or self._info_val_str("options_metavar")
),
add_help_option=add_help_option,
no_args_is_help=no_args_is_help,
hidden=hidden,
Expand Down Expand Up @@ -272,7 +274,7 @@ def add_typer(
help: Optional[str] = Default(None),
epilog: Optional[str] = Default(None),
short_help: Optional[str] = Default(None),
options_metavar: str = Default("[OPTIONS]"),
options_metavar: Optional[str] = None,
add_help_option: bool = Default(True),
hidden: bool = Default(False),
deprecated: bool = Default(False),
Expand All @@ -294,7 +296,7 @@ def add_typer(
help=help,
epilog=epilog,
short_help=short_help,
options_metavar=options_metavar,
options_metavar=options_metavar or self.info.options_metavar,
add_help_option=add_help_option,
hidden=hidden,
deprecated=deprecated,
Expand Down Expand Up @@ -325,6 +327,12 @@ def __call__(self, *args: Any, **kwargs: Any) -> Any:
)
raise e

def _info_val_str(self, name: str) -> str:
val = getattr(self.info, name)
val_str = val.value if isinstance(val, DefaultPlaceholder) else val
assert isinstance(val_str, str)
return val_str


def get_group(typer_instance: Typer) -> TyperGroup:
group = get_group_from_info(
Expand Down
19 changes: 10 additions & 9 deletions typer/rich_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,7 @@ def _get_rich_console(stderr: bool = False) -> Console:
)


def _make_rich_rext(
def _make_rich_text(
*, text: str, style: str = "", markup_mode: MarkupMode
) -> Union[Markdown, Text]:
"""Take a string, remove indentations, and return styled text.
Expand Down Expand Up @@ -190,20 +190,21 @@ def _get_help_text(
help_text = help_text.partition("\f")[0]

# Get the first paragraph
first_line = help_text.split("\n\n")[0]
first_line, *remaining_paragraphs = help_text.split("\n\n")

# Remove single linebreaks
if markup_mode != MARKUP_MODE_MARKDOWN and not first_line.startswith("\b"):
first_line = first_line.replace("\n", " ")
yield _make_rich_rext(
yield _make_rich_text(
text=first_line.strip(),
style=STYLE_HELPTEXT_FIRST_LINE,
markup_mode=markup_mode,
)

# Get remaining lines, remove single line breaks and format as dim
remaining_paragraphs = help_text.split("\n\n")[1:]
if remaining_paragraphs:
if markup_mode != MARKUP_MODE_RICH:
yield Text("")
if markup_mode not in (MARKUP_MODE_RICH, MARKUP_MODE_MARKDOWN):
# Remove single linebreaks
remaining_paragraphs = [
x.replace("\n", " ").strip()
Expand All @@ -217,7 +218,7 @@ def _get_help_text(
# Join with double linebreaks if markdown
remaining_lines = "\n\n".join(remaining_paragraphs)

yield _make_rich_rext(
yield _make_rich_text(
text=remaining_lines,
style=STYLE_HELPTEXT,
markup_mode=markup_mode,
Expand Down Expand Up @@ -272,7 +273,7 @@ def _get_parameter_help(
for x in paragraphs
]
items.append(
_make_rich_rext(
_make_rich_text(
text="\n".join(paragraphs).strip(),
style=STYLE_OPTION_HELP,
markup_mode=markup_mode,
Expand Down Expand Up @@ -331,7 +332,7 @@ def _make_command_help(
paragraphs[0] = paragraphs[0].replace("\n", " ")
elif paragraphs[0].startswith("\b"):
paragraphs[0] = paragraphs[0].replace("\b\n", "")
return _make_rich_rext(
return _make_rich_text(
text=paragraphs[0].strip(),
style=STYLE_OPTION_HELP,
markup_mode=markup_mode,
Expand Down Expand Up @@ -674,7 +675,7 @@ def rich_format_help(
# Remove single linebreaks, replace double with single
lines = obj.epilog.split("\n\n")
epilogue = "\n".join([x.replace("\n", " ").strip() for x in lines])
epilogue_text = _make_rich_rext(text=epilogue, markup_mode=markup_mode)
epilogue_text = _make_rich_text(text=epilogue, markup_mode=markup_mode)
console.print(Padding(Align(epilogue_text, pad=False), 1))


Expand Down
Loading