Skip to content

Use CommonMarkConverter#1

Closed
szepeviktor wants to merge 1 commit intotwigphp:3.xfrom
szepeviktor:patch-1
Closed

Use CommonMarkConverter#1
szepeviktor wants to merge 1 commit intotwigphp:3.xfrom
szepeviktor:patch-1

Conversation

@szepeviktor
Copy link
Copy Markdown

@szepeviktor szepeviktor commented Aug 10, 2019

@fabpot Try running phpstan analyze -l max src/


namespace Twig\Extra\Markdown;

use League\CommonMark\CommonMarkConverter;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Actually, the issue is the constructor which should have used MarkdownExtra instead. I've fixed on twigphp/twig as this is a read-only repo. Thank you.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

:)

@fabpot fabpot closed this Aug 10, 2019
@szepeviktor szepeviktor deleted the patch-1 branch August 10, 2019 13:08
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