Skip to content

Conversation

@dreamwasp
Copy link
Contributor

@dreamwasp dreamwasp commented Oct 22, 2025

Overview

Converts the production workflows from CircleCI to GHA including cacheing. Also configures required checks from CircleCI to Github settings

PR Checklist

  • Related to designs:
  • Related to JIRA ticket: GMT-287
  • I have run this code to verify it works
  • This PR includes unit tests for the code change
  • This PR includes testing instructions tests for the code change
  • The alpha package of this PR is passing end-to-end tests in all relevant Codecademy repositories

Testing Instructions

I don't really know how to test this. I can make the test below fail to show you the required tests are failing but I do think this is a yolo situation.

I will merge this despite the CircleCi Pipeline failure, I'll disable the pipeline after this is shipped.

PR Links and Envs

N/A

@nx-cloud
Copy link

nx-cloud bot commented Oct 22, 2025

View your CI Pipeline Execution ↗ for commit e8d3517


☁️ Nx Cloud last updated this comment at 2025-10-23 16:28:10 UTC

Comment on lines -23 to -31
- name: Wait for CI build
uses: fountainhead/[email protected]
id: wait-for-build
with:
token: ${{ secrets.ACTIONS_GITHUB_TOKEN }}
checkName: build-test
ref: ${{ github.event.pull_request.head.sha || github.sha }}
timeoutSeconds: 1200
intervalSeconds: 30
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Image

changed the required checks to the following, so shouldn't need this anymore

Copy link
Contributor Author

@dreamwasp dreamwasp Oct 23, 2025

Choose a reason for hiding this comment

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

& pr-title

@@ -0,0 +1,156 @@
# CircleCI to GitHub Actions Migration Summary
Copy link
Contributor Author

@dreamwasp dreamwasp Oct 23, 2025

Choose a reason for hiding this comment

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

will remove before shiupping, but wanted to keep cursors notes for the review

@dreamwasp dreamwasp changed the title robo start, need to review ci: convert production workflows Oct 23, 2025
@codecademydev
Copy link
Collaborator

📬 Published Alpha Packages:

@codecademy/[email protected]
@codecademy/[email protected]

@github-actions
Copy link
Contributor

@dreamwasp dreamwasp marked this pull request as ready for review October 23, 2025 17:14
@dreamwasp dreamwasp requested a review from a team as a code owner October 23, 2025 17:14
- main

env:
NODE_VERSION: '22.13.1'
Copy link
Member

Choose a reason for hiding this comment

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

minor but i don't think this is used, we use the version from .nvmrc when we run actions/setup-node

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