Migrate from CircleCI to GitHub Actions #5654
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
canova
force-pushed
the
github-actions-migration
branch
from
fb66a3d to
691cdc9
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #5654 +/- ##
=======================================
Coverage 85.63% 85.63%
=======================================
Files 312 312
Lines 30867 30867
Branches 8490 8490
=======================================
Hits 26434 26434
Misses 4009 4009
Partials 424 424 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Member
Author
|
Looks like this PR is working well. I think we can do the switch. The GitHub actions mostly improved the running times of the tasks slightly but not all: After: |
canova
changed the title
This dependency was only used inside the CircleCI task for uploading the code coverage results with `yarn codecov`. Since we use the official codecov action in GitHub Actions now, we don't really need it anymore. Note that running `yarn codecov` locally is not possible as it will error out since we are not in a CI environment.
canova
force-pushed
the
github-actions-migration
branch
from
691cdc9 to
051e627
Compare
Member
Author
|
Updated the settings as well. The migration is complete, and it looks like it's successful! |
Merged
canova
added a commit
that referenced
this pull request
Nov 12, 2025
Changes: [Nazım Can Altınova] Use getEmptySourceTable instead of constructing the table manually (#5641) [Florian Quèze] Source view for marker stacks (#5633) [Nazım Can Altınova] Add fallback to funcTable.lineNumber when computing line timings (#5644) [Florian Quèze] Show the marker keys instead of the labels in marker tooltips when alt is pressed. (#5625) [Francesco Lodolo] Add GitHub workflow to check Fluent reference files (#5647) [Nazım Can Altınova] Switch from Gitpod to GitHub Codespaces (#5653) [Nazım Can Altınova] Migrate from CircleCI to GitHub Actions (#5654) [Nazım Can Altınova] Run the CI in the production branch and on every pull request (#5661) [Nazım Can Altınova] Convert l10n-sync.js from Flow comment types to JSDoc comments for Typescript (#5659) [Florian Quèze] Display correct unit in Stack Chart tooltips for size profiles. (#5658) [Florian Quèze] Hide track management UI in single track profiles. (#5656) [Florian Quèze] Rename hiddenTrackCount to trackCount. (#5665) [Florian Quèze] Cleanup the profile info panel for size profiles (show file size and name, hide empty sections). (#5657) [Markus Stange] Use valibot for symbolication API response validation (#5666) [Nazım Can Altınova] Make sure to fetch the full git history for l10n-sync job in CI (#5667) [Nazım Can Altınova] Do not switch to call tree when clicking the activity graph while on sample based panels (#5672) And thanks to our localizers: be: Mikalai Udodau de: Michael Köhler el: Jim Spentzos en-GB: Ian Neal es-CL: ravmn fr: Théo Chevalier fy-NL: Fjoerfoks ia: Melo46 it: Francesco Lodolo [:flod] nl: Mark Heijl pt-BR: Marcelo Ghelman ru: berry, Valery Ledovskoy sv-SE: Andreas Pettersson tr: Grk zh-CN: Olvcpr423, wxie zh-TW: Pin-guang Chen
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR attempts to migrate from CircleCI to GitHub.
There are some more things left to do:
It looks like the workflows are working well. I added a new ssh key for l10n sync but there is no straightforward way to test the l10n sync for now because it's a cron job. I think it's fine and we can check it in the following morning once this PR is merged.
Things to do after we merge this PR: