-
Notifications
You must be signed in to change notification settings - Fork 8
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
New actions fail on repo with fork #98
Comments
I think we have to pass the token into the action explicitly (No idea why it worked during my tests...). I will create a quick patch for main. |
Thanks @assignUser. With #99, still no luck. Now seems an unrelated {pak} problem. https://github.com/r-lib/styler/runs/4828086477?check_suite_focus=true. |
I had that error with simstudy the last days, usually running the job again
works. I will open an issue in the actions repo.
|
ok, will try re-run. |
Thanks for the issue 👍 . |
And it failed again https://github.com/r-lib/styler/runs/4828631586?check_suite_focus=true... Hope this can be fixed or it's just by chance. |
Ugh, that's annoying... I think it's by chance maybe some request times out? They are using RSPM for dependency resolution afaik so maybe somethings spotty there? I'll add that run to the issue. |
It works! 🎉 r-lib/styler#868 (comment) |
Indeed, it did 🥳. Let's see how it goes with the {pak} error in the future. I previously started using a more recent RSPM as you suggested. |
In {styler}, I can't seem to make the action pass after replacing the Workflows with the templates from {touchstone}:
https://github.com/r-lib/styler/actions/runs/1699543116
Any idea why? There is a corresponding PR that shows passing but the comment from {touchstone} is missing. From the error message, it seems like the secrets context is missing.
The text was updated successfully, but these errors were encountered: