Skip to content

Conversation

@eleftherias
Copy link
Member

Had a discussion with @lujunsan about some alternatives, but ultimately decided to stick to the original approach. Adding a summary of the discussion for future reference.

@codecov
Copy link

codecov bot commented Sep 11, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 42.62%. Comparing base (b838fe3) to head (e8b3286).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1834      +/-   ##
==========================================
- Coverage   42.64%   42.62%   -0.02%     
==========================================
  Files         184      184              
  Lines       21852    21852              
==========================================
- Hits         9319     9315       -4     
- Misses      11820    11823       +3     
- Partials      713      714       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@coveralls
Copy link
Collaborator

Coverage Status

coverage: 39.965% (-0.02%) from 39.982%
when pulling e8b3286 on groups-registry-alt
into 5341e33 on main.

@eleftherias eleftherias merged commit da647cf into main Sep 11, 2025
18 of 19 checks passed
@eleftherias eleftherias deleted the groups-registry-alt branch September 11, 2025 10:53
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.

4 participants