Skip to content

Remove CircleCI config #12737

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

Merged
merged 2 commits into from
Jun 10, 2025
Merged

Remove CircleCI config #12737

merged 2 commits into from
Jun 10, 2025

Conversation

silv-io
Copy link
Member

@silv-io silv-io commented Jun 10, 2025

Motivation

This is the final PR of the initiative to switch to GitHub Actions. It removes the old CircleCI config, since everything now runs off of GitHub actions:

  • Main tests in PRs and main branch
  • MA/MR tests in PRs and main branch
  • Pushing of dev releases
  • Pushing of container images

Changes

  • Remove CircleCI config.yml

Testing

Since the Webhook triggers got disabled, there should not be any error of a missing CircleCI config in the checks.

@silv-io silv-io added the semver: minor Non-breaking changes which can be included in minor releases, but not in patch releases label Jun 10, 2025
@silv-io silv-io requested a review from thrau as a code owner June 10, 2025 12:08
Copy link
Member

@alexrashed alexrashed left a comment

Choose a reason for hiding this comment

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

Nice! This is the final step in unifying our CI / using GitHub Actions everywhere! 🥳

Copy link

Test Results - Preflight, Unit

21 595 tests  ±0   19 940 ✅ ±0   6m 23s ⏱️ +4s
     1 suites ±0    1 655 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit 708ede8. ± Comparison against base commit 3a5f08b.

Copy link

Test Results (amd64) - Acceptance

7 tests  ±0   5 ✅ ±0   3m 10s ⏱️ -1s
1 suites ±0   2 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit 708ede8. ± Comparison against base commit 3a5f08b.

@silv-io silv-io added the review: merge when ready Signals to the reviewer that a PR can be merged if accepted label Jun 10, 2025
Copy link

Test Results - Alternative Providers

987 tests   584 ✅  23m 36s ⏱️
  4 suites  403 💤
  4 files      0 ❌

Results for commit 708ede8.

Copy link

Test Results (amd64) - Integration, Bootstrap

    5 files      5 suites   2h 21m 12s ⏱️
5 229 tests 4 300 ✅ 929 💤 0 ❌
5 235 runs  4 300 ✅ 935 💤 0 ❌

Results for commit 708ede8.

Copy link

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   1h 45m 21s ⏱️ + 2m 3s
4 872 tests ±0  4 095 ✅ ±0  777 💤 ±0  0 ❌ ±0 
4 874 runs  ±0  4 095 ✅ ±0  779 💤 ±0  0 ❌ ±0 

Results for commit 708ede8. ± Comparison against base commit 3a5f08b.

@alexrashed alexrashed merged commit f6075f6 into master Jun 10, 2025
42 checks passed
@alexrashed alexrashed deleted the turn-off-circleci branch June 10, 2025 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review: merge when ready Signals to the reviewer that a PR can be merged if accepted semver: minor Non-breaking changes which can be included in minor releases, but not in patch releases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants