-
Notifications
You must be signed in to change notification settings - Fork 45
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
Draft: Update pFUnit to use v2 template interfaces #464
Conversation
Sorry, I got pulled aside to other things and haven't gotten back to this. @tclune please review... |
Same here. Vacation combined with official travel. I have the ticket pegged in my inbox. Hopefully can review sometime this week. Quite jet-lagged at the moment though. |
Understood. Whenever you are able to get to it is fine. |
Looks like the CI is looking for an fArgParse hash that is in Goddard-Fortran-Ecosystem/fArgParse#142 but isn't in develop yet. |
Yeah - should have read the "conversation" rather than just staring at the changes. I've approved the fArgParse PR, but you'll need to at least touch the ChangeLog for CI to be happy there. If the branch were local I would have done that for you. |
I've created a new PR that brings in your changes to fArgParse. Closing this one. |
This adds the changes needed for pFUnit to build using the v2 versions of the template interfaces.
Note that this PR depends on Goddard-Fortran-Ecosystem/fArgParse#142 and will not build without it.
Resolves #463.