Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

change MPI_QUANTITIES to enum class #2279

Merged
merged 7 commits into from
May 14, 2024
Merged

Conversation

bigfooted
Copy link
Contributor

Proposed Changes

Give a brief overview of your contribution here in a few sentences.
change MPI_QUANTITIES to enum class

  • I am submitting my contribution to the develop branch.
  • My contribution generates no new compiler warnings (try with --warnlevel=3 when using meson).
  • My contribution is commented and consistent with SU2 style (https://su2code.github.io/docs_v7/Style-Guide/).
  • I used the pre-commit hook to prevent dirty commits and used pre-commit run --all to format old commits.
  • I have added a test case that demonstrates my contribution, if necessary.
  • I have updated appropriate documentation (Tutorials, Docs Page, config_template.cpp), if necessary.

Common/include/option_structure.hpp Outdated Show resolved Hide resolved
Common/include/option_structure.hpp Outdated Show resolved Hide resolved
@bigfooted bigfooted merged commit 83cac12 into develop May 14, 2024
34 of 35 checks passed
@bigfooted bigfooted deleted the enumclass_mpi_quantities branch May 14, 2024 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants