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
doc: Repair changelog link #958
Conversation
Good catch, thanks. I wonder if rather than pointing out an url, if it would be possible to refer to a file, I know this possible but I always forget the syntax to use... a kink of thing like this:
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #958 +/- ##
=====================================
Coverage 56% 56%
=====================================
Files 89 89
Lines 9767 9767
Branches 1819 1819
=====================================
+ Hits 5477 5479 +2
+ Misses 3920 3919 -1
+ Partials 370 369 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There's
for referring to other pages within the Sphinx project, but the Unless your build script involves a step that copies the Also: Linking directly to the github URL means people get to see the most up to date version. |
Indeed, I just tested it locally and it finished with a display of the passed url...
No I don't think we want to spend more time on this topic, using the url will do the job.
Indeed, thanks. |
Thank you for your contribution. Much appreciated |
Thanks for the review, and for your work on this project! |
Currently, the link to the changelog found in the Eventlet docs at https://eventlet.readthedocs.io/en/latest/index.html#changelog links to the Python changelog, at https://docs.python.org/3/whatsnew/changelog.html#changelog.
This PR changes it to point to https://github.com/eventlet/eventlet/blob/master/NEWS
Other possibilities
I noticed there's a reference to a
changelog.html
here:eventlet/doc/source/real_index.html
Line 151 in c22b896
so maybe there was some intention to generate a true changelog page, belonging to the docs? Don't know.