-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 RTD links to additional tools in the README #10079
Fix RTD links to additional tools in the README #10079
Conversation
Thank you for catching this and fixing it @bnjmnp. Do you mind adding a redirect here too please ? Lines 60 to 98 in 614f80e
|
I don't fully understand the redirects dict. I assume in this case it would be this:
|
Yeam seems sensible. It's possible to test on the readthedoc instance generated from the PR : for example : https://pylint--10079.org.readthedocs.build/en/10079/how_tos/index.html redirect as expected). In this case: |
No luck with my new commit, the links still don't work. |
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.
Thank you @bnjmnp !
It was my pleasure. |
Type of Changes
Description
Just a quick fix of the README