-
Notifications
You must be signed in to change notification settings - Fork 42
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 node12 instead of Docker to make it work in MacOS environment #3
Comments
@edvinasbartkus: was unaware of this limitation, will investigate, thanks. Also curious if this alternative might work for you (I haven't attempted anything similar so I have no idea how and if that will works): We split the worklow in 3 dependent jobs:
What do you think? Though I am unaware how to implement dependent jobs within the same workflow |
@ambientlight good point about dependent jobs! will try to do that. that might work. thanks |
@edvinasbartkus: sure, I will keep this open until I have more feedback on reimplementing this in |
I'm also very interested in this because I'm attempting to run e2e tests for an iOS app that integrates with an AWS AppSync backend (incidentally using @edvinasbartkus's great guide here would you believe). The intention I have is to use a common CI environment initially so I won't be creating and destroying CloudFormation stacks, but I will need to do the equivalent of I was hoping instead to use your github action so later as it looks great, but if it can't run on MacOS because of the use of Docker, that's a non-starter for now. I know I could just run |
Also for reference (at least for others like myself who wasn't aware):
For this particular action we rely on the There may be a better way, but at least doing it that way means we're less reliant on the internals of An alternative could be to follow a similar approach to this GitHub Action - peter-evans/create-pull-request - which supports running on Mac. It ends up running Python via Anyway just thought I'd add this as I was curious so did some digging, and some of it may help. Comments, improvements, and corrections are most welcome! |
@danrivett: yes sure, I do see a good point, I have looked into it few month back, reimplementing in node is not complex actually, and this amplify action is quite trivial to begin with, hopefully will be able to find a timeframe to do this soon, thanks for the references! |
Just ran into this, any updates? |
I had a go at this, which you can see at https://github.com/johnf/amplify-cli-action The javascript code is done and simple. The tricky part is getting amplify to actually run. You can't install global packages and getting amplify to run out of the node_modules directory where it is installed got a bit tricky. |
If I try to use this step with macos-latest, I get the following (I want to run integration tests for iOS app):
Maybe it's possible to use
node12
instead of Dockerfile, like in https://github.com/actions/setup-node/blob/master/action.yml#L20The text was updated successfully, but these errors were encountered: