Skip to content

Add a failing test about diff code highlighting #124

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

Conversation

javiereguiluz
Copy link
Collaborator

@javiereguiluz javiereguiluz commented Oct 20, 2021

@wouterj reported that a code block here https://symfony.com/doc/current/security.html#form-login didn't show any highlighting.

It's true. The problem occurs when a diff code block doesn't start its first line with + or - In this failing test, the first code block works but the second doesn't (to be completely honest, the first one doesn't work either because of some white space issue which I can't see, but it mostly works) <-- this problem was fixed by Ryan. Thanks!

@weaverryan
Copy link
Contributor

Fixed at: doctrine/rst-parser#168

@javiereguiluz javiereguiluz force-pushed the failing_code_diff_test branch from f5c759d to 9c45a43 Compare October 22, 2021 10:39
@javiereguiluz javiereguiluz force-pushed the failing_code_diff_test branch from 9c45a43 to ac48cde Compare October 22, 2021 10:43
@javiereguiluz javiereguiluz merged commit 22d0822 into symfony-tools:main Oct 22, 2021
@javiereguiluz javiereguiluz deleted the failing_code_diff_test branch October 22, 2021 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants