Skip to content

Conversation

@ehuss
Copy link
Contributor

@ehuss ehuss commented Nov 8, 2025

This includes several commits to fix various markdown errors in the rfc book. This is in preparation for mdbook 0.5 which is a little more sensitive to markdown errors (mostly generating warnings, or not rendering as expected). This should be compatible with both 0.5 and 0.4.

r? internal-sites

Rendered

ehuss added 8 commits November 7, 2025 18:23
This fixes the unclosed `<a>` tags.
This ensures the code block is part of the list.
This fixes some bare HTML tags by wrapping them in a code span or using
html entities.
In mdbook 0.5, it will support definition lists, and unfortunately
the way this is written gets interpreted as a definition list.
The simple fix is to remove the unneeded newline.

See pulldown-cmark/pulldown-cmark#1057
This close tag was missing the slash.
`img` is a void element and does not take an end tag.
I assume this is what is intended. Otherwise the `simp` footnote
definition is unused.
In commonmark, markup is not translated inside raw HTML. This makes a
few changes to make this process correctly.
@ehuss ehuss added the not-rfc For PRs that fix things like spelling mistakes, wrong file names, etc. label Nov 8, 2025
Copy link
Member

@Urgau Urgau left a comment

Choose a reason for hiding this comment

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

LGTM.

## Use cases

### Poor interaction with overloadable deref and index <a name=overload>
### Poor interaction with overloadable deref and index { #overload }
Copy link
Member

Choose a reason for hiding this comment

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

I didn't knew this was a syntax, very fancy.

@Urgau Urgau merged commit 131858b into rust-lang:master Nov 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

not-rfc For PRs that fix things like spelling mistakes, wrong file names, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants