Skip to content

fix(ci): use existing unit group in Coveralls workflow - #6417

Merged
rubencarvalho merged 2 commits into
mainfrom
fix/coveralls-ci-group
Jun 18, 2026
Merged

fix(ci): use existing unit group in Coveralls workflow#6417
rubencarvalho merged 2 commits into
mainfrom
fix/coveralls-ci-group

Conversation

@aramos-adobe

@aramos-adobe aramos-adobe commented Jun 17, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

  • Fixes the Coveralls CI job failing with Could not find any group named coveralls-ci
  • The coveralls-ci group was removed during the monorepo migration (feat: introduce dual-generation monorepo architecture #5785) but the workflow still referenced it
  • Switches to the existing unit group, matching the CircleCI coverage job

Test plan

  • Locally verified --group unit resolves 144 test files (old --group coveralls-ci fails immediately)
  • Coveralls workflow passes on this PR

@aramos-adobe @cursoragent
fix(ci): use existing unit group in Coveralls workflow
62548e8 
The coveralls-ci test group was removed during the monorepo migration
but the workflow still referenced it, causing CI to fail immediately.

Co-authored-by: Cursor <cursoragent@cursor.com>
@aramos-adobe
aramos-adobe requested a review from a team as a code owner June 17, 2026 19:15
@changeset-bot

changeset-bot Bot commented Jun 17, 2026 •
edited
Loading

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 6dd54ae

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@aramos-adobe aramos-adobe self-assigned this Jun 17, 2026
@coveralls

coveralls commented Jun 17, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Coverage Report for CI Build 27759887189

Warning

No base build found for commit 0325754 on main.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 96.24%

Details

  • Patch coverage: No coverable lines changed in this PR.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →

Coverage Stats

Coverage Status
Relevant Lines: 39108
Covered Lines: 37837
Line Coverage: 96.75%
Relevant Branches: 6450
Covered Branches: 6008
Branch Coverage: 93.15%
Branches in Coverage %: Yes
Coverage Strength: 458.36 hits per line
💛 - Coveralls

@github-actions

github-actions Bot commented Jun 17, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

📚 Branch Preview Links

🔍 First Generation Visual Regression Test Results

When a visual regression test fails (or has previously failed while working on this branch), its results can be found in the following URLs:

Deployed to Azure Blob Storage: pr-6417

If the changes are expected, update the current_golden_images_cache hash in the circleci config to accept the new images. Instructions are included in that file.
If the changes are unexpected, you can investigate the cause of the differences and update the code accordingly.

caseyisonit
caseyisonit approved these changes Jun 17, 2026
View reviewed changes

@caseyisonit caseyisonit left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

:shipit:

@rubencarvalho rubencarvalho added the run_vrt Triggers the Chromatic VRT run for 2nd-gen label Jun 17, 2026
@aramos-adobe aramos-adobe added Status:Ready for review PR ready for review or re-review. and removed run_vrt Triggers the Chromatic VRT run for 2nd-gen labels Jun 17, 2026
@rubencarvalho
rubencarvalho enabled auto-merge (squash) June 17, 2026 19:43
@rubencarvalho rubencarvalho added the run_vrt Triggers the Chromatic VRT run for 2nd-gen label Jun 17, 2026
@rubencarvalho
rubencarvalho merged commit f3e1c17 into main Jun 18, 2026
31 checks passed
@rubencarvalho
rubencarvalho deleted the fix/coveralls-ci-group branch June 18, 2026 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rubencarvalho rubencarvalho rubencarvalho approved these changes
@caseyisonit caseyisonit caseyisonit approved these changes

Assignees

@aramos-adobe aramos-adobe

Labels

run_vrt Triggers the Chromatic VRT run for 2nd-gen Status:Ready for review PR ready for review or re-review.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@aramos-adobe @coveralls @rubencarvalho @caseyisonit