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

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

Closed
kellertuer opened this issue Jul 19, 2021 · 2 comments · Fixed by #1017
Closed

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

kellertuer opened this issue Jul 19, 2021 · 2 comments · Fixed by #1017

Comments

@kellertuer
Copy link

See here
JuliaManifolds/Manifolds.jl#400 (comment)
where the code change included markdown, but since posted as a markdown comment, this got screwed up.

@kellertuer kellertuer changed the title Reviewdog gets confused when proposing changed of code that include markdown Reviewdog gets confused when proposing changes of code that include markdown Jul 19, 2021
@shogo82148
Copy link
Contributor

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
Copy link
Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants