-
Notifications
You must be signed in to change notification settings - Fork 258
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: Merged duplicate README and index sections #1205
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #1205 +/- ##
==========================================
- Coverage 92.16% 92.15% -0.01%
==========================================
Files 97 97
Lines 12332 12334 +2
Branches 2534 2534
==========================================
+ Hits 11366 11367 +1
Misses 645 645
- Partials 321 322 +1
... and 6 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
Okay, so it looks like the reason things ended up like this is that there's no way to unify reference links like .. _appendix of the manual: ./legal.html#license
.. _installation: ./installation.html#installation And README.rst could have: .. _appendix of the manual: https://nipy.org/nibabel/legal.html#license
.. _installation: https://nipy.org/nibabel/installation.html#installation It's not great, but we don't really reorganize much. |
Sorry if I'm missing something, but couldn't we just use the second option anywhere there's a * :ref:`User Documentation <manual>` (manual) with: * `User Documentation <manual>`_ (manual) and include: .. _manual: https://nipy.org/nibabel/manual.html#manual in Also, seems to me like the content in parentheses under the Documentation heading could be removed for a cleaner look. EDIT: Actually, where do you find this to be a problem? I don't see anything wrong looking at my local build. |
The reason not to use an absolute URL in |
@effigies please see the proposed changes. Hope I understood correctly. |
Resolves #1204.