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

Skipping code blocks and fixing docs build. #47

Merged
merged 1 commit into from
Nov 20, 2021

Conversation

Robpol86
Copy link
Contributor

While working on another PR I ran into this bug on my test website. I had a test document with only an HTML code block and found out unescaped HTML was added to the meta tag:

<meta property="og:description" content="<meta property="og:image" content="_images/normal.jpg" /> <img alt="_images/normal.jpg" src="_images/normal.jpg" />" />
<meta property="og:image" content="_images/normal.jpg" />

Fixing this by skipping literal blocks.

Also fixed docs build. I was getting the exception: AttributeError: 'Values' object has no attribute 'section_self_link'

This was caused by sphinx-doc/sphinx#9825. Updating dependencies fixed the issue.

While working on another PR I ran into this bug on my test website. I
had a test document with only an
HTML code block and found out unescaped HTML was added to the meta tag:

```html
<meta property="og:description" content="<meta property="og:image" content="_images/normal.jpg" /> <img alt="_images/normal.jpg" src="_images/normal.jpg" />" />
<meta property="og:image" content="_images/normal.jpg" />
```

Fixing this by skipping literal blocks.

Also fixed docs build. I was getting the exception:
`AttributeError: 'Values' object has no attribute 'section_self_link'`

This was caused by sphinx-doc/sphinx#9825.
Updating dependencies fixed the issue.
@Daltz333 Daltz333 merged commit e20f753 into wpilibsuite:main Nov 20, 2021
@Robpol86 Robpol86 deleted the skip_code_block branch November 20, 2021 04:55
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 this pull request may close these issues.

2 participants