-
Notifications
You must be signed in to change notification settings - Fork 439
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
Change source links to github #4113
Conversation
@jorgepiloto and @Revathyvenugopal162, you may consider making the changes implemented here into a general purpose sphinx extension as I think it's quite useful to link to github. |
Codecov Report
@@ Coverage Diff @@
## main #4113 +/- ##
=======================================
Coverage 95.56% 95.56%
=======================================
Files 94 95 +1
Lines 20213 20246 +33
=======================================
+ Hits 19316 19349 +33
Misses 897 897 |
@MatthewFlamm, seems that #4111 indeed fixed codecov. |
Co-authored-by: Tetsuo Koyama <tkoyama010@gmail.com>
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.
LGTM
* add linkcode * Update pyvista/utilities/docs.py Co-authored-by: Tetsuo Koyama <tkoyama010@gmail.com> --------- Co-authored-by: Tetsuo Koyama <tkoyama010@gmail.com>
* bump to 0.38.4 * update req in earnest * fix plotter reopening (#4102) * Update testing-and-deployment.yml * Update requirements_test.txt Co-authored-by: MatthewFlamm <39341281+MatthewFlamm@users.noreply.github.com> * Update testing-and-deployment.yml * Apply suggestions from code review * Change source links to github (#4113) * add linkcode * Update pyvista/utilities/docs.py Co-authored-by: Tetsuo Koyama <tkoyama010@gmail.com> --------- Co-authored-by: Tetsuo Koyama <tkoyama010@gmail.com> * Fix Plotter docs (#3968) --------- Co-authored-by: MatthewFlamm <39341281+MatthewFlamm@users.noreply.github.com> Co-authored-by: Tetsuo Koyama <tkoyama010@gmail.com> Co-authored-by: Bane Sullivan <bane.sullivan@kitware.com>
The
mne
libray usessphinx.ext.linkcode
rather thansphinx.ext.viewcode
and they link directly to github:mne.io.read_raw
This PR implements the same logic and uses a source link to GitHub. It's much cleaner than having source within our documentation, and it has the (minor) added advantage of decreasing the build time and size of our docs.