Skip to content

Conversation

jorgepiloto
Copy link
Member

@jorgepiloto jorgepiloto commented Apr 12, 2023

This pull-request introduces all the required changes and updates to pass the technical review of the project according to the PyAnsys guidelines.

Changes include:

  • Renaming of the repository from pyDyna to pydyna
  • Updates on the project metadata
    • Author and maintainer is ANSYS, Inc.
    • Contact e-mails point to PyAnsys core team
    • Add AUTHORS.md
  • Update pre-commit hooks
  • Point codespell to the accept.txt file used by Vale

@github-actions github-actions bot added the maintenance Generic maintenance related label Apr 12, 2023
@RobPasMue
Copy link
Member

@Revathyvenugopal162 @jorgepiloto ... can we have only one PR for this?...

@jorgepiloto
Copy link
Member Author

Sure @RobPasMue, we are merging #145 against this one.

@RobPasMue
Copy link
Member

Thanks @jorgepiloto - I will wait to review this one in that case.

wenhuiuy and others added 3 commits April 12, 2023 14:51
* maint: update precommit

* maint: update labeler

* doc: update contributning

* Apply suggestions from code review

Co-authored-by: Jorge Martínez <28702884+jorgepiloto@users.noreply.github.com>

* Fix: add labeller for dependency

* Doc: fix the header

* Doc: fix the vale error

* Doc: fix the vale error for sentance case

* maint: remove the proto files

* Debug: try to remove codecov dependency

---------

Co-authored-by: Jorge Martínez <28702884+jorgepiloto@users.noreply.github.com>
@github-actions github-actions bot added dependencies Related with project dependencies documentation Improvements or additions to documentation labels Apr 12, 2023
@RobPasMue
Copy link
Member

AUTHORS.md file is missing!

Revathyvenugopal162 and others added 3 commits April 13, 2023 15:50
Co-authored-by: Roberto Pastor Muela <37798125+RobPasMue@users.noreply.github.com>
@jorgepiloto jorgepiloto marked this pull request as ready for review June 22, 2023 09:20
@jorgepiloto jorgepiloto requested a review from a team June 22, 2023 09:38
Copy link
Member

@RobPasMue RobPasMue left a comment

Choose a reason for hiding this comment

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

LGTM @jorgepiloto! Some minor requests

@jorgepiloto jorgepiloto merged commit 13667cf into main Jun 23, 2023
@jorgepiloto jorgepiloto deleted the maint/tech-review branch June 23, 2023 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Related with project dependencies documentation Improvements or additions to documentation maintenance Generic maintenance related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants