Skip to content

Conversation

@germa89
Copy link
Collaborator

@germa89 germa89 commented May 6, 2024

@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

@germa89 germa89 changed the title Clarifying API Clarifying MAPDL commands API section May 6, 2024
@codecov-commenter
Copy link

codecov-commenter commented May 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.16%. Comparing base (f6224a5) to head (6c66c2b).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3071      +/-   ##
==========================================
- Coverage   86.64%   84.16%   -2.49%     
==========================================
  Files          52       52              
  Lines        9553     9553              
==========================================
- Hits         8277     8040     -237     
- Misses       1276     1513     +237     

@germa89 germa89 self-assigned this May 6, 2024
@germa89 germa89 requested a review from clatapie May 6, 2024 17:40
@germa89
Copy link
Collaborator Author

germa89 commented May 6, 2024

@Revathyvenugopal162 you might want to add the pyansys logos with transparencies to the ansys-sphinx-theme package.

@germa89 germa89 enabled auto-merge (squash) May 6, 2024 18:12
@germa89
Copy link
Collaborator Author

germa89 commented May 7, 2024

@pyansys-ci-bot LGTM.

Copy link
Contributor

@pyansys-ci-bot pyansys-ci-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Approving this PR because germa89 said so in here 😬

LGTM

@germa89 germa89 merged commit dc0a56c into main May 7, 2024
@germa89 germa89 deleted the fix/mapdl-api-fix branch May 7, 2024 08:12
@clatapie clatapie changed the title Clarifying MAPDL commands API section docs: clarifying MAPDL commands API section Jun 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants