-
Notifications
You must be signed in to change notification settings - Fork 36
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
Use updated ghactions4r workflow in r4ss #982
Conversation
This branch had some attempts to work on another issue but those were reverted and all the commits will just be squashed when merged. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@e-perl-NOAA, thanks for working on this.
The changes look good (based on my rudimentary understanding of yml files), but it's still failing on the macOS-latest. Any idea why?
It's still failing because of the change to the get_ss3_exe() function to not use tokens in the gh::gh() call. With github actions, the same runner for mac is used so it calls the api enough times that the github api gets mad. I have no idea how to fix it for github actions. I tried several things but didn't get anywhere. See here for a more eloquent explanation of what is going on. |
The fix in #986 should have this PR work fine now, but it looks like we should change the |
Sorry @iantaylor-NOAA I didn't realize that I hadn't changed that back to main but that's why we have these reviews! |
It might still fail because it's using the github action from this branch, not the one from main. I would need to rebase this branch to get it passing all the checks which I can do if you would like. |
See issue #981.