-
Notifications
You must be signed in to change notification settings - Fork 122
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
refactor: annotate pymapdl part 1 #3569
base: main
Are you sure you want to change the base?
Conversation
Thanks for opening a Pull Request. If you want to perform a review write a comment saying: @ansys-reviewer-bot review |
@ansys-reviewer-bot review |
Okay, I will trigger a review of your PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Co-authored-by: Camille <[email protected]>
... so? |
@ansys-reviewer-bot review |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #3569 +/- ##
==========================================
+ Coverage 86.97% 87.15% +0.17%
==========================================
Files 187 187
Lines 14623 14627 +4
==========================================
+ Hits 12719 12748 +29
+ Misses 1904 1879 -25 |
@RobPasMue any news about the bot? |
@ansys-reviewer-bot review |
Okay, I will trigger a review of your PR. |
Nope - too many parallel tracks currently |
Roger that... you tell me... this PR is no priority and if you think it is a good test for the bot (since it is just annotating... no other logic)... I can keep it open... or go ahead... up to you. |
Go ahead... don't wait for me. Sorry @germa89 |
No big deal at all! :) |
Description
Annotate some files
Issue linked
Related to #3480
Checklist
draft
if it is not ready to be reviewed yet.feat: adding new MAPDL command
)