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

Use updated ghactions4r workflow in r4ss #982

Merged
merged 17 commits into from
Dec 10, 2024
Merged

Conversation

e-perl-NOAA
Copy link
Contributor

See issue #981.

@e-perl-NOAA
Copy link
Contributor Author

This branch had some attempts to work on another issue but those were reverted and all the commits will just be squashed when merged.

Copy link
Contributor

@iantaylor-NOAA iantaylor-NOAA left a 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?

@e-perl-NOAA
Copy link
Contributor Author

e-perl-NOAA commented Dec 2, 2024

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.

@iantaylor-NOAA
Copy link
Contributor

The fix in #986 should have this PR work fine now, but it looks like we should change the @add-additional-args-to-yaml to back to @main now that the branch has been merged in nmfs-fish-tools/ghactions4r#147, right?

@e-perl-NOAA
Copy link
Contributor Author

Sorry @iantaylor-NOAA I didn't realize that I hadn't changed that back to main but that's why we have these reviews!

@e-perl-NOAA
Copy link
Contributor Author

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.

@iantaylor-NOAA iantaylor-NOAA merged commit b307e29 into main Dec 10, 2024
6 checks passed
@iantaylor-NOAA iantaylor-NOAA deleted the test-updates-ghactions4r branch December 10, 2024 19:02
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.

2 participants