search

ansys / pyaedt

AEDT Python Client Package
https://aedt.docs.pyansys.com
MIT License
201 stars 120 forks source link

DOCS: Fix title to follow google headings #4863

Closed SMoraisAnsys closed 3 months ago

SMoraisAnsys commented 3 months ago

Previous PR on extension seem to have added a warning that does not pass the CICD anymore. This PR should fix the typo problem

ansys-reviewer-bot[bot] commented 3 months ago

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

@ansys-reviewer-bot review

PipKat commented 3 months ago

@SMoraisAnsys The proper name for the app is HFSS 3D Layout. Perhaps try adding all three words as one line entry to Vale's accept list? The heading would then be HFSS 3D Layout extensions.

codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 81.95%. Comparing base (11f03cc) to head (cf87751).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #4863 +/- ## ========================================== - Coverage 83.14% 81.95% -1.20% ========================================== Files 118 118 Lines 54080 54080 ========================================== - Hits 44967 44321 -646 - Misses 9113 9759 +646 ```
SMoraisAnsys commented 3 months ago

@SMoraisAnsys The proper name for the app is HFSS 3D Layout. Perhaps try adding all three words as one line entry to Vale's accept list? The heading would then be HFSS 3D Layout extensions.

Yeah I though that all of them where added but we were missing 3D... I also added 2D to avoid future issues and removed full expressions.