Skip to content

Feature/publish gh actions#128

Merged
meganrm merged 5 commits intomasterfrom
feature/publish-gh-actions
May 24, 2021
Merged

Feature/publish gh actions#128
meganrm merged 5 commits intomasterfrom
feature/publish-gh-actions

Conversation

@meganrm
Copy link
Copy Markdown
Contributor

@meganrm meganrm commented May 20, 2021

Problem

Move deployment from Jenkins to github actions
Link to story or ticket

Solution

I copied over jobs from CFE, with one change: CFE just copies the staging build into the production bucket, but since we want a different backend for staging and production, the production script does a fresh build

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update

Thanks for contributing!

@@ -0,0 +1,68 @@
name: Auto deploy staging AWS
env:
NODE_VERSION: '12.x'
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I keep forgetting, why are we stuck on node 12? gh-actions workers have node 14 something auto installed

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's the version we were using when we started the cfe repo I think, we can update it. For a while we had both 12 and 14. I think it's pretty standard to include 2 so you can feel comfortable going up to the next version.

@meganrm meganrm requested review from blairlyons and schoinh May 20, 2021 23:14
@schoinh
Copy link
Copy Markdown
Contributor

schoinh commented May 20, 2021

Probably unrelated, but what's daily-new-viewer.yml in the github workflows folder? Is that in use? If not should it go away?

Copy link
Copy Markdown
Contributor

@schoinh schoinh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me... This is gonna be great! Thanks!

@meganrm meganrm merged commit ed867a4 into master May 24, 2021
@meganrm meganrm deleted the feature/publish-gh-actions branch June 8, 2021 17:44
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