Skip to content

[Console] Fix codeblock markup #18937

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

Merged
merged 1 commit into from
Oct 2, 2023
Merged

[Console] Fix codeblock markup #18937

merged 1 commit into from
Oct 2, 2023

Conversation

smnandre
Copy link
Member

Bug visible on 5.4 6.2 6.3 6.4 & 7.0

5.4 6.2 6.4
a b c

@@ -585,7 +585,7 @@ call ``setAutoExit(false)`` on it to get the command result in ``CommandTester``
When testing your commands, it could be useful to understand how your command
reacts on different settings like the width and the height of the terminal.
You have access to such information thanks to the
:class:`Symfony\\Component\\Console\\Terminal` class:
:class:`Symfony\\Component\\Console\\Terminal` class::
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we spot this automatically @alamirault

Any idea?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can't or it will be very hard.

Malformed Literal blocks can be confused with a lot other RST blocks.

Following regex can be a start .+:\n\n , but we must exclude a lot of false positive

@OskarStark
Copy link
Contributor

Thanks for fixing this bug Simon.

@OskarStark OskarStark merged commit f679796 into symfony:5.4 Oct 2, 2023
@smnandre smnandre deleted the patch-1 branch November 25, 2023 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants