docs: add new APDL resource link to learning and update external links#4067
docs: add new APDL resource link to learning and update external links#4067
Conversation
|
Thanks for opening a Pull Request. If you want to perform a review write a comment saying: @ansys-reviewer-bot review |
Reviewer's GuideThis PR enriches the documentation by adding a new APDL resource entry to the learning guide and refreshing several external links to their current URLs. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Hey @germa89 - I've reviewed your changes - here's some feedback:
- Add the new APDL post to the learning section’s toctree so it appears in the sidebar navigation.
- Include a brief description for the APDL resource in learning.rst to provide context instead of only a link.
- Check that the external links in links.rst adhere to the project’s reference link style and update any inconsistencies.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Add the new APDL post to the learning section’s toctree so it appears in the sidebar navigation.
- Include a brief description for the APDL resource in learning.rst to provide context instead of only a link.
- Check that the external links in links.rst adhere to the project’s reference link style and update any inconsistencies.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
There was a problem hiding this comment.
Pull Request Overview
This PR adds a new APDL learning resource to the “Recommended articles” section and reorganizes the external links in links.rst.
- Added an APDL article reference in
learning.rst - Resorted and updated external link anchors in
links.rst
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| doc/source/links.rst | Alphabetized and updated external link definitions |
| doc/source/getting_started/learning.rst | Added APDL learning link under Recommended articles |
Comments suppressed due to low confidence (1)
doc/source/getting_started/learning.rst:205
- Indent the new APDL bullet and its accompanying link to match the existing styling (two spaces before the
*and four spaces before the link line).
* APDL: The Language of Simulation.
|
@pyansys-ci-bot LGTM. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4067 +/- ##
=======================================
Coverage 89.02% 89.03%
=======================================
Files 187 187
Lines 14993 14993
=======================================
+ Hits 13348 13349 +1
+ Misses 1645 1644 -1 🚀 New features to boost your workflow:
|

Description
Add a post about APDL.
Issue linked
NA
Checklist
draftif it is not ready to be reviewed yet.feat: adding new MAPDL command)Summary by Sourcery
Add a new APDL learning resource link and update external links in documentation
Documentation: