-
Notifications
You must be signed in to change notification settings - Fork 2
ci: update ci_cd.yml to use ubuntu latest #291
Conversation
|
@klmcadams If I'm wrong in doing those changes, please let me know and close the PR |
|
@SMoraisAnsys this change is fine! I just pinned it to ubuntu-22.04 because of pymapdl, but if they're moving to use ubuntu-latest then I don't see a problem updating it here as well |
|
Ha... I just realized PyMAPDL hasn't fixed this yet... pinging @germa89 |
|
I'm in the process of deprecating this library in favor of ansys-tools-common, is there any update for this PR? Should I migrate this changes directly to here? https://github.com/ansys/ansys-tools-common/blob/72ba917b3db36095ed8658f51df63192ee600396/.github/workflows/run_mapdl_tests.yml#L26-L30 |
|
Anything done here should be migrated to ansys-tools-common @AlejandroFernandezLuces Also, make sure that we are not missing any changes from this package on the ansys-tools-common one |
|
Closing, moved to ansys/ansys-tools-common#90 |
Following the changes performed in #268, this PRs applies a change that will be required soon since ansys/pymapdl#3814 is going to be merged.