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

Convert chapters 15 #3956

Merged
merged 2 commits into from
Jun 14, 2024
Merged

Convert chapters 15 #3956

merged 2 commits into from
Jun 14, 2024

Conversation

bzierk
Copy link
Contributor

@bzierk bzierk commented Jun 12, 2024

Adds Listing conversion for chapter 12, 13, 14, and 15

@chriskrycho
Copy link
Contributor

Thanks! Note that there are real CI failures due to some missed opening/closing tags, etc., so once you address that, I'll be happy to review it and get it landed!

@bzierk bzierk changed the title Convert chapters 12-15 Convert chapters 15 Jun 14, 2024
@bzierk
Copy link
Contributor Author

bzierk commented Jun 14, 2024

The opening/closing tags is a bad error, I've found. This is the result of the < characters causing parsing Token::Attribute to fail. It seems like the error doesn't actually get propagated up but the <Listing> never gets created after that error so mdbook thinks its a missing tags issue. I opened another PR to get 12-14 in and I'll keep working on the issue with xmlparser and <.

@asggWa

This comment was marked as off-topic.

@chriskrycho
Copy link
Contributor

Ah, that makes sense (per your comment on the tracking issue) @bzierk – thanks!

@chriskrycho chriskrycho merged commit 3216c0f into rust-lang:main Jun 14, 2024
4 of 6 checks passed
chriskrycho added a commit that referenced this pull request Oct 15, 2024
This restores the work originally done in #3956, which had to be dropped
from that PR because this chapter was blocked on XML handling, which was
resolved in #3975.

Co-authored-by: Bryan Zierk <[email protected]>
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.

3 participants