Skip to content

Conversation

@mbareford
Copy link
Contributor

…on multiple nodes.

@mbareford
Copy link
Contributor Author

Nektar++ is tested from a module load and also from a locally built nektar++.
There is some duplication across test_module_nektarpp.py and test_nektarpp.py - perhaps this can be avoided.

Copy link
Contributor

@aturner-epcc aturner-epcc left a comment

Choose a reason for hiding this comment

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

@mbareford Thanks for the PR. Can you check the linting/syntax tests that are failing and modify the PR so these are passed please?

@mbareford
Copy link
Contributor Author

@aturner-epcc The lint tests now pass.

@aturner-epcc aturner-epcc merged commit 0a73917 into EPCCed:main May 29, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants