-
Notifications
You must be signed in to change notification settings - Fork 57
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
(Temporarily) fix stable docs link in README.md #249
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #249 +/- ##
=======================================
Coverage 81.53% 81.53%
=======================================
Files 18 18
Lines 1587 1587
=======================================
Hits 1294 1294
Misses 293 293 ☔ View full report in Codecov by Sentry. |
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.
yeah it's been bugging me too
This PR has to be reverted. The new link points to completely outdated docs. IMO a broken link is better than one to incorrect docs. |
This reverts commit 31016f4.
Okay, but is there a link to the current stable docs? Sorry for merging but I thought this was the right one too, given that it is close to the current URL. So I have in fact used the orange site as a stable docs |
It's some docs from years ago. The link was removed on purpose in #246. |
#246 pointed out that the docs are not correctly deployed. Unfortunately, this is still the case so in the meantime a quick "bandaid" fix which makes the stable docs accessible is just to change the stable docs link.
Right now the stable docs are hard to find unless you know that removing
/stable
from the url goes there.Apologies if this is the wrong fix to the problem, it was just bugging me.