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

chore: Use GitHub Actions for CI #259

Merged
merged 10 commits into from
Sep 29, 2021
Merged

chore: Use GitHub Actions for CI #259

merged 10 commits into from
Sep 29, 2021

Conversation

akash1810
Copy link
Member

@akash1810 akash1810 commented Sep 20, 2021

What does this change?

Switches CI from TeamCity to GitHub Actions (with the use of https://github.com/guardian/actions-assume-aws-role).

There are a few benefits to using GHA individually:

  • No build queue
  • No need to perform a dance with nvm
  • No need to globally install yarn
  • Build logs are co-located with the PR (via the Checks tab)

Collectively, use of GHA means we don't have to maintain our own CI infrastructure.

How to test

devx::prism should appear as a deployable project in Riff-Raff. It should have two files: the cloudformation and the deb and it should be successfully deployed to CODE.

Have we considered potential risks?

Deployment gets a little tricky as the Riff-Raff project name has changed and we all have to remember this. Riff-Raff restrictions help here though.

build.sbt Outdated Show resolved Hide resolved
sihil
sihil previously approved these changes Sep 28, 2021
Copy link
Contributor

@sihil sihil left a comment

Choose a reason for hiding this comment

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

This is great. I agree that we should default the buckets in the plugin. Do you want to do that before telling everyone?

build.sbt Outdated Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
This action should pick up the node version from the `.nvmrc` file.
@akash1810
Copy link
Member Author

akash1810 commented Sep 28, 2021

This PR is almost at the end! Just waiting for Maven to show the latest release of sbt-riffraff-plugin. Done!

Copy link
Contributor

@sihil sihil left a comment

Choose a reason for hiding this comment

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

🎉

@akash1810 akash1810 merged commit 71d03ba into main Sep 29, 2021
@akash1810 akash1810 deleted the aa-gha branch September 29, 2021 08:42
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