Removed duplicate installation instructions from API Docs#4927
Removed duplicate installation instructions from API Docs#4927orbeckst merged 5 commits intoMDAnalysis:developfrom
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Hello there first time contributor! Welcome to the MDAnalysis community! We ask that all contributors abide by our Code of Conduct and that first time contributors introduce themselves on GitHub Discussions so we can get to know you. You can learn more about participating here. Please also add yourself to package/AUTHORS as part of this PR.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #4927 +/- ##
========================================
Coverage 93.42% 93.42%
========================================
Files 177 177
Lines 21859 21859
Branches 3078 3078
========================================
Hits 20422 20422
Misses 986 986
Partials 451 451 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
RMeli
left a comment
RMeli
left a comment
There was a problem hiding this comment.
If I'm not mistaken, this is your first contribution in this repo. Please add yourself to package/AUTHORS.
|
Hi @orbeckst, thanks for your comment! I have addressed it and made some additional refinements to the page. |
orbeckst
left a comment
orbeckst
left a comment
There was a problem hiding this comment.
Can you please check that you don't need to backtick-quote rst anchors with spaces or special characters?
| :maxdepth: 4 | ||
| :caption: Documentation | ||
| :numbered: | ||
| :numbered: |
package/doc/sphinx/source/index.rst
Outdated
| **Source code** is available from | ||
| https://github.com/MDAnalysis/mdanalysis/ and is packaged under the | ||
| `Lesser GNU Public Licence, version 3 or any later version`_. Individual components | ||
| The MDAnalysis source code is available on [GitHub](https://github.com/MDAnalysis/mdanalysis/) and is |
There was a problem hiding this comment.
this link is the only one not rendering correctly in markdown
|
@micaela-matta can you have a quick look, please? Once you approve it will auto-merge. |
github-project-automation
bot
moved this from In Progress
to Done
in Streamlining Documentation (SDG23)
* Redesign installation guide in User Guide * fixes #402 * fixes #405 * see also MDAnalysis/mdanalysis#4927 --------- Co-authored-by: Oliver Beckstein <orbeckst@gmail.com>
…#4927) * Fixes MDAnalysis#4929 * Removed duplicate installation instructions from API Docs * related to MDAnalysis/UserGuide#404
5 participants
Fixes #4929
Changes made in this Pull Request:
PR Checklist
package/CHANGELOGfile updated?package/AUTHORS? (If it is not, add it!)Developers Certificate of Origin
I certify that I can submit this code contribution as described in the Developer Certificate of Origin, under the MDAnalysis LICENSE.
📚 Documentation preview 📚: https://mdanalysis--4927.org.readthedocs.build/en/4927/