Skip to content

feat: solution submodule #3881

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

Open
wants to merge 11 commits into
base: feat/main_commands
Choose a base branch
from
Open

Conversation

clatapie
Copy link
Contributor

Description

This PR follows the PyConverter-XML2Py integration plan to automate the PyMAPDL_commands documentation.
The changes have been generated using pyconverter-xml2py and more specifically mapdl-cmd-conv.

This PR focus on the solution submodule.

You can check the mapdl-cmd-conv documentation to have a look at the output of each submodules.

Pinging @ansys/pymapdl-developers for visibility. Feel free to provide any feedback on the way the docstrings and the source code generation are handled.

Issue linked

This PR is meant to be merged within the feat/main_commands branch. The latter will gather all the submodule changes, one by one, prior to be merged to the main branch.

Checklist

@clatapie clatapie self-assigned this Apr 30, 2025
@clatapie clatapie requested a review from a team as a code owner April 30, 2025 12:47
@clatapie clatapie requested review from germa89 and pyansys-ci-bot and removed request for a team April 30, 2025 12:47
@ansys-reviewer-bot
Copy link
Contributor

Thanks for opening a Pull Request. If you want to perform a review write a comment saying:

@ansys-reviewer-bot review

@github-actions github-actions bot added documentation Documentation related (improving, adding, etc) new feature Request or proposal for a new feature labels Apr 30, 2025
@clatapie clatapie changed the title feat: adding solution submodule feat: adding solution submodule Apr 30, 2025
@clatapie clatapie changed the title feat: adding solution submodule feat: solution submodule Apr 30, 2025
@clatapie clatapie requested a review from RobPasMue April 30, 2025 16:23
@clatapie
Copy link
Contributor Author

All the unit tests seems to be passing. This PR is ready to be reviewed @germa89 and @RobPasMue.

@clatapie
Copy link
Contributor Author

clatapie commented May 5, 2025

Last commit fixed the error raised by Codacy Static Code.

@germa89 germa89 added the DO NOT MERGE Not ready to be merged yet label May 14, 2025
@germa89
Copy link
Collaborator

germa89 commented May 14, 2025

As discussed internally, we are leaving this PR and #3872 for later. So I am adding the DO NOT MERGE label.

@germa89 germa89 mentioned this pull request May 14, 2025
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DO NOT MERGE Not ready to be merged yet documentation Documentation related (improving, adding, etc) new feature Request or proposal for a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants