Skip to content

Conversation

@ryanmrichard
Copy link
Member

@ryanmrichard ryanmrichard commented Nov 18, 2024

Is this pull request associated with an issue(s)?
No

Description
This PR refactors the source so that:

  • Tests relying on other plugins are integration tests.
  • Executables relying on other plugins (or dependencies of other plugins) are optional.

TODOs

  • Setup CI to test the optional features.

@ryanmrichard ryanmrichard self-assigned this Nov 18, 2024
@ryanmrichard ryanmrichard marked this pull request as ready for review November 18, 2024 18:22
@ryanmrichard
Copy link
Member Author

I turned off the checking the TAMM integration tests. The library builds with TAMM support, but I get a runtime error in the integration tests related to the MPI communicator. I don't feel like spending more time debugging this at the moment.

@ryanmrichard ryanmrichard merged commit 8593759 into master Nov 19, 2024
@jwaldrop107
Copy link
Member

🚀 [bumpr] Bumped! New version:v0.0.3 Changes:v0.0.2...v0.0.3

@ryanmrichard ryanmrichard deleted the drivers branch November 19, 2024 15:06
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.

3 participants