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

Move CI to Github Actions #63

Merged
merged 1 commit into from
Nov 6, 2020
Merged

Move CI to Github Actions #63

merged 1 commit into from
Nov 6, 2020

Conversation

simonihmig
Copy link
Contributor

In preparation for a future Embroider support PR that is expected to add Embroider test scenarios, I wanted to switch to an ember-try based setup. And when doing that, I figured we can move to Github Actions at the same time.

The workflow is based on what my usual setup looks like, but feel free to suggest changes if you want it aligned to other projects or so...

@simonihmig
Copy link
Contributor Author

Oh, right. @ef4 seems you have to enable GH actions for this repo! (If you agree with the switch here that is)

@simonihmig simonihmig mentioned this pull request Nov 6, 2020
@ef4
Copy link
Owner

ef4 commented Nov 6, 2020

Hmm, they're already allowed. I'll just merge it and see if that kicks it off.

@ef4 ef4 merged commit c35a41d into ef4:master Nov 6, 2020
@simonihmig simonihmig deleted the ghactions branch November 7, 2020 11:19
@simonihmig
Copy link
Contributor Author

Yeah, they ran successfully now on master luckily. It makes sense actually, as kicking off new (unapproved) workflows from a fork (giving them access to secrets) would be a security risk. IIRC I have read about related limitations...

@mansona
Copy link
Collaborator

mansona commented Nov 11, 2020

any secrets are not accessible at all which causes a few problems when working with github actions 🙈 but the limitation here is that you need to commit an empty workflow file to master before PRs will start building 👍

thanks for doing this work btw 🎉

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.

3 participants