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

fix: GitHub changed markdown rendering for headings/anchors #3408

Closed
wants to merge 1 commit into from

Conversation

MikeRalphson
Copy link
Member

@MikeRalphson MikeRalphson commented Oct 19, 2023

Closes #3207 #3160 #3136 #3128 #3029 #3197 #3132

This PR mechanically corrects for the change made by GitHub to its markdown renderer which no longer honours the mixed markdown/html form:

### <a name="myAnchor"></a> My Anchor

(the use of name or id makes no difference), and only links to the implicit anchor #my-anchor now work.

This PR affects the main branch, so published versions of the OAS only. If merged, follow-up PRs will be raised for in-flight -dev branch versions.

An example of the result of this PR can be previewed here.

Signed-off-by: Mike Ralphson <mike.ralphson@gmail.com>
@handrews
Copy link
Member

@MikeRalphson could you rebase this and see if various javascript updates fixed the runtime checks?

@handrews
Copy link
Member

handrews commented Feb 2, 2024

Superseded by #3548 in order to get the PR checks to pass. Mike is still the credited author of the commit in the new PR.

@handrews handrews closed this Feb 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants