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
Close #9016: linkcheck builder failed to check the anchors of github.com #9260
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is quite useful to link to a section of GitHub rendered markup. I think the benefits outweights the maintenance burden.
Perhaps add a test for the rewriting, e.g. linking to https://github.com/sphinx-doc/sphinx#documentation?
That way, we’ll know if GitHub changes their prefix.
@francoisfreitag Thank you for your opinion. Okay, I'll continue to implement this. |
bf14d3a
to
92335bd
Compare
Just updted! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Otherwise LGTM, thanks!
Co-authored-by: François Freitag <mail@franek.fr>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great, thanks!
Thank you for reviewing! |
Dear Takeshi and François, thanks a stack for bringing in this excellent improvement. With kind regards, |
Feature or Bugfix
Purpose