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

bumping cmake version to 3.19, needed to ensure HDF5::HDF5 target exists #416

Merged
merged 1 commit into from
Mar 28, 2023

Conversation

peter-d
Copy link
Contributor

@peter-d peter-d commented Mar 27, 2023

Fixes #407

@ghost
Copy link

ghost commented Mar 27, 2023

Did you mean to say it fixes #407 ?

@peter-d
Copy link
Contributor Author

peter-d commented Mar 27, 2023

My bad - should be #414

@ghost ghost assigned peter-d Mar 27, 2023
@ghost ghost added enhancement Enhancement or request component: sparta Issue is related to sparta framework component: plato issues related to Plato (part of Helios) labels Mar 27, 2023
@ghost
Copy link

ghost commented Mar 27, 2023

I don't see issue with this -- we're using cmake 3.25 in house. Let me do a little testing just to make sure and I'll merge it in! Thanks!

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

LGTM

@peter-d
Copy link
Contributor Author

peter-d commented Mar 28, 2023

Just to double check: what's the policy here? Am I allowed to merge myself once approved?

@ghost
Copy link

ghost commented Mar 28, 2023

Just to double check: what's the policy here? Am I allowed to merge myself once approved?

Yep. If I've given that power, you are free to use it. 😁

@peter-d peter-d merged commit 6f0636f into master Mar 28, 2023
@peter-d peter-d deleted the peter-d/issue_414_cmake_version branch March 28, 2023 14:26
github-actions bot pushed a commit that referenced this pull request Mar 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: plato issues related to Plato (part of Helios) component: sparta Issue is related to sparta framework enhancement Enhancement or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

WeightedContextCounter: move from examples to sparta proper
1 participant