refactor: annotate pymapdl part 1#3569
Conversation
germa89
requested review from
clatapie and
pyansys-ci-bot
and removed request for
a team
|
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. |
Co-authored-by: Camille <78221213+clatapie@users.noreply.github.com>
... so? |
|
@ansys-reviewer-bot review |
…mapdl into refactor/annotate-files
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #3569 +/- ##
==========================================
+ Coverage 87.01% 87.15% +0.14%
==========================================
Files 187 187
Lines 14661 14660 -1
==========================================
+ Hits 12757 12777 +20
+ Misses 1904 1883 -21 |
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| ) # match number groups | ||
| BC_REGREP = re.compile(r"^\s*([0-9]+)\s*([A-Za-z]+)((?:\s+[0-9]*[.]?[0-9]+)+)$") | ||
| BC_REGREP: re.Pattern = re.compile( | ||
| r"^\s*([0-9]+)\s*([A-Za-z]+)((?:\s+[0-9]*[.]?[0-9]+)+)$" |
Check failure
Code scanning / CodeQL
Inefficient regular expression
|
@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! :) |
5 participants
Description
Annotate some files
Issue linked
Related to #3480
Checklist
draftif it is not ready to be reviewed yet.feat: adding new MAPDL command)