search

reviewdog / reviewdog

🐶 Automated code review tool integrated with any code analysis tools regardless of programming language
https://medium.com/@haya14busa/reviewdog-a-code-review-dog-who-keeps-your-codebase-healthy-d957c471938b#.8xctbaw5u
MIT License
7.94k stars 422 forks source link

Reviewdog gets confused when proposing changes of code that include markdown #999

Closed kellertuer closed 3 years ago

kellertuer commented 3 years ago

See here https://github.com/JuliaManifolds/Manifolds.jl/pull/400#discussion_r672321087 where the code change included markdown, but since posted as a markdown comment, this got screwed up.

shogo82148 commented 3 years ago

The document says it is needed to wrap them inside quadruple backticks.

https://docs.github.com/en/github/writing-on-github/working-with-advanced-formatting/creating-and-highlighting-code-blocks#fenced-code-blocks

To display triple backticks in a fenced code block, wrap them inside quadruple backticks.

```` 
```
Look! You can see my backticks.
```
````

Look! You can see my backticks.

kellertuer commented 3 years ago

Thanks, that could help. But what if the code uses quadruple backticks? Then watchdog has to use quintruple ones?