Skip to content

Conversation

@JeffGarland
Copy link
Member

This PR covers links to all the current issues linked from nbballots. I took as slightly different style than a few of the recent issues that just put bold text into the description -- here I made it a note and put the link to the repo. I didn't modify some of the ones that already had these 'text references'.

other notes: I notice the mdash, hellip, and sect give errors from xmllint. These might need to be added to the dtd?

@jwakely
Copy link
Member

jwakely commented Oct 25, 2025

other notes: I notice the mdash, hellip, and sect give errors from xmllint. These might need to be added to the dtd?

They're all in the DTD already, are you telling xmllint to use the DTD?

The relevant option is --dtdvalid xml/lwg-issue.dtd

Or just use bin/lint.sh in the repo which checks everything.

@jwakely
Copy link
Member

jwakely commented Oct 25, 2025

The "Validate XML" CI workflow use bin/lint.sh too, so pull requests are checked.

@JeffGarland
Copy link
Member Author

other notes: I notice the mdash, hellip, and sect give errors from xmllint. These might need to be added to the dtd?

They're all in the DTD already, are you telling xmllint to use the DTD?

The relevant option is --dtdvalid xml/lwg-issue.dtd

No - naively I was thinking xmllint would pick them up from header if running in the xml directory.

<!DOCTYPE issue SYSTEM "lwg-issue.dtd">

Or just use bin/lint.sh in the repo which checks everything.

roger that. ironically I just pulled out xmllint bc it's what installed on the mac without even thinking of looking in bin.

@jwakely jwakely merged commit e1ce0e1 into cplusplus:master Oct 27, 2025
2 checks passed
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