-
Notifications
You must be signed in to change notification settings - Fork 21
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
TEST PR: confirming new .workflows/*yml changes working as expected in GitHub runner #239
Conversation
…st them also modify workflow/*yml files to test in GitHub on pull_request only, and not also on push
also reverted workflow/main_new.yml, for now at least
Not sure what you are doing in this PR. Code coverage is working in the develop branch currently, I just checked with a test PR. |
This I realize your recent yml changes are working in the updated |
Are you still having problems with gcovr? It is working in the develop branch. (Not for python - is that what you are trying to achieve?) |
No, I'm not still having problems with gcovr. That went away once I stopped trying to run it on the The issue I'm having now is with one of the modified test codes (the last one, in fact) failing on MacOS. But it only fails on MacOS and nowhere else, and I don't have a MacOS platform of my own to run this on locally and hook up a debugging tool, so I have to resort to pushing up commits with temporary print statements and use the GitHub runner to try to sort out what's going on here. |
Now, that this is working, I'll open a formal PR to merge this branch and close the underlying issue. |
see NOAA-EMC/NCEPLIBS-grib_util#165 for more details of the impetus behind this change