Skip to content

Migrate CI to Github Actions#699

Open
zetter-rpf wants to merge 11 commits intomainfrom
cursor/github-actions-ci-migration-bc0d
Open

Migrate CI to Github Actions#699
zetter-rpf wants to merge 11 commits intomainfrom
cursor/github-actions-ci-migration-bc0d

Conversation

@zetter-rpf
Copy link
Contributor

@zetter-rpf zetter-rpf commented Feb 27, 2026

Status

What's changed?

Replace our Circle CI integration with Github actions.

My motivation for doing this now was a flakey test in https://github.com/RaspberryPiFoundation/digital-editor-issues/issues/1144

It's possible that the different test environment may reduce flakyness.

Steps to perform after deploying to production

cursoragent and others added 9 commits February 27, 2026 11:35
Co-authored-by: Chris Zetter <zetter-rpf@users.noreply.github.com>
Co-authored-by: Chris Zetter <zetter-rpf@users.noreply.github.com>
Co-authored-by: Chris Zetter <zetter-rpf@users.noreply.github.com>
Co-authored-by: Chris Zetter <zetter-rpf@users.noreply.github.com>
Co-authored-by: Chris Zetter <zetter-rpf@users.noreply.github.com>
Co-authored-by: Chris Zetter <zetter-rpf@users.noreply.github.com>
Co-authored-by: Chris Zetter <zetter-rpf@users.noreply.github.com>
Co-authored-by: Chris Zetter <zetter-rpf@users.noreply.github.com>
Co-authored-by: Chris Zetter <zetter-rpf@users.noreply.github.com>
@cla-bot
Copy link

cla-bot bot commented Feb 27, 2026

We require contributors to sign our Contributor License Agreement, and we don't have you on file. In order for us to review and merge your code, please complete this form and we'll get you added and review your contribution as soon as possible.

Co-authored-by: Chris Zetter <zetter-rpf@users.noreply.github.com>
@cla-bot
Copy link

cla-bot bot commented Feb 27, 2026

We require contributors to sign our Contributor License Agreement, and we don't have you on file. In order for us to review and merge your code, please complete this form and we'll get you added and review your contribution as soon as possible.

@github-actions
Copy link

github-actions bot commented Feb 27, 2026

Test coverage

89.07% line coverage reported by SimpleCov.
Run: https://github.com/RaspberryPiFoundation/editor-api/actions/runs/22494466969

@zetter-rpf zetter-rpf marked this pull request as ready for review February 27, 2026 13:50
Copilot AI review requested due to automatic review settings February 27, 2026 13:50
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request migrates the CI/CD infrastructure from CircleCI to GitHub Actions. The change is motivated by organizational shift away from CircleCI and a desire to potentially reduce test flakiness through a different test environment. The migration is comprehensive, removing all CircleCI artifacts and replacing them with equivalent GitHub Actions workflows.

Changes:

  • Replaced CircleCI configuration with a GitHub Actions workflow that runs linting and tests
  • Implemented custom bash scripts for test coverage reporting via PR comments
  • Updated development environment configuration by removing CircleCI-specific extensions
  • Updated documentation to reflect the new CI platform

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
AGENTS.md Updated CI documentation to reference GitHub Actions with Ruby 3.4, Postgres 12, Redis
.github/workflows/ci.yml New CI workflow with lint and test jobs, including postgres and redis services, coverage reporting
.github/scripts/post_coverage_comment.sh New script to post or update coverage comments on PRs using GitHub API
.github/scripts/build_coverage_comment.sh New script to parse SimpleCov coverage data and format it for PR comments
.devcontainer/devcontainer.json Removed CircleCI VSCode extension from development container
.circleci/record_coverage Deleted legacy CircleCI coverage recording script
.circleci/config.yml Deleted legacy CircleCI configuration file

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@cla-bot cla-bot bot added the cla-signed label Feb 27, 2026
@adrian-rpf adrian-rpf self-requested a review February 27, 2026 16:27
Copy link
Contributor

@adrian-rpf adrian-rpf 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.
Just the question on if you think the EDITOR_ENCRYPTION_KEY should be excluded.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migrate CI from CircleCI to Github Actions - Editor-API

4 participants