ember-learn/ember-cli-addon-docs

Out of the box GitHub Actions support

buschtoens opened this issue · 10 comments

It would be great to get out of the box support for GitHub Actions or at least a go-to guide.

@buschtoens I would be glad to work with you on this. We have a lot we need to do to modernize this project. I've been slowly chipping away at it.

I got it working in ember-intl/ember-intl#1425, without the use of ember-cli-deploy-git-ci, thanks to the help of @dfreeman.

I was trying to base my setup on @NullVoxPopuli's setup in ember-jsqr, where he is using ember-cli-deploy-git-ci with a DEPLOY_KEY env, but couldn't get it to work for the life of me. Turns out the DEPLOY_KEY secret I had set on GitHub was missing the final \n, which caused ssh-add to fail (dfreeman/ember-cli-deploy-git-ci#11).

Anyhow, the DEPLOY_KEY-less solution is superior anyway IMO.

The only annoying thing is that the credentials are set via git config --local, so they aren't available in the repo that is cloned by ember-cli-deploy-git. I had to manually steal them, which is definitely not very pretty.

Unless a really nice ember-cli-deploy pattern / plugin for GitHub Actions emerges, I'm inclined to say that we should ditch it in favor of a GitHub Action (for deploying to GitHub pages that is).

It would be nice if we had a CI agnostic solution, so things could be built and deployed on Travis, GitHub Actions, Circle etc. but if it is easier to just support GitHub Actions, I do think that is the best choice to lock down on.

Definitely, CI-agnostic would be best - I agree. We don't necessarily need to drop ember-cli-deploy in order to adopt GitHub Actions. We could maintain support for both, but it would mean double work.

Ideally we'd have a nice GitHub Actions plugin for ember-cli-deploy, but I doubt that's easily possible.

@buschtoens certainly open to suggestions. I think one defined path is okay here, and removing dependencies is always nice, however what about the case where a user wants to deploy adhoc?

Looks like there is another issue for this too #437

👋 just wanted to say that I've been updating ember-jsqr a bit, and the GH Workflow I had for addon-docs didn't work anymore (used old / no-longer-existing / insecure GH Action API). I removed that bit and now only docs updates on the master branch are deploying (no versioned docs). I'm investigating a bit still

Turns out I had a [skip ci]-type behavior on my deploy of tags. oops. NullVoxPopuli/ember-jsqr@7812771

nope, that wasn't it. It seems my tagged workflow isn't triggering at all 🤔