Skip to content

Don't register dependency groups until they're succesfully added#387

Merged
nathanjmcdougall merged 2 commits intomainfrom
385-revertdont-add-a-dependency-group-to-defaults-if-creating-it-fails
Mar 5, 2025
Merged

Don't register dependency groups until they're succesfully added#387
nathanjmcdougall merged 2 commits intomainfrom
385-revertdont-add-a-dependency-group-to-defaults-if-creating-it-fails

Conversation

@nathanjmcdougall
Copy link
Copy Markdown
Collaborator

No description provided.

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Mar 5, 2025

CodSpeed Performance Report

Merging #387 will not alter performance

Comparing 385-revertdont-add-a-dependency-group-to-defaults-if-creating-it-fails (e8afc9d) with main (74ae4d9)

Summary

✅ 1 untouched benchmarks

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 5, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.14%. Comparing base (74ae4d9) to head (e8afc9d).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #387      +/-   ##
==========================================
+ Coverage   98.10%   98.14%   +0.03%     
==========================================
  Files          70       70              
  Lines        3008     3013       +5     
==========================================
+ Hits         2951     2957       +6     
+ Misses         57       56       -1     

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

@nathanjmcdougall nathanjmcdougall merged commit 40c7ff8 into main Mar 5, 2025
15 checks passed
@nathanjmcdougall nathanjmcdougall deleted the 385-revertdont-add-a-dependency-group-to-defaults-if-creating-it-fails branch March 5, 2025 03:15
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.

Revert/don't add a dependency group to defaults if creating it fails

2 participants