-
Notifications
You must be signed in to change notification settings - Fork 889
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
Move to github actions #792
Conversation
This works on my fork: https://github.com/autrilla/sops/runs/1622187193 |
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.
Awesome, thank you so much @autrilla!
@ajvb can you merge this manually? Right now we need TravisCI to pass, but since they ended support for open source, it never will. |
@autrilla Looks like our settings our not allowing the There might be a wait to add this to an allow list, will look into it today. |
It should be fairly easy to replace it with rustup if getting it allowed is
not trivial.
…On Wed, 6 Jan 2021 at 17:49, AJ Bahnken ***@***.***> wrote:
@autrilla <https://github.com/autrilla> Looks like our settings our not
allowing the actions-rs action to be ran -
https://github.com/mozilla/sops/actions/runs/466628788
There might be a wait to add this to an allow list, will look into it
today.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#792 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AARH4V6EGWKYDRHR7TZEIHTSYSIATANCNFSM4VNJMYTQ>
.
|
@autrilla yeah, I'd say let's go with rustup for now. The process (https://wiki.mozilla.org/GitHub#GitHub_Actions) is there, but will require some back and forth. If you want me to initiate getting |
Basically reopening #576 now that Travis CI is deprecated (#787). It doesn't handle releases right now, but if it at least means we have working tests for PRs and so on, I think that's a net win.
Fixes #787.