Skip to content

Conversation

@moliholy
Copy link
Collaborator

This PR removes the unused dependency sp-weights.

@codecov
Copy link

codecov bot commented Aug 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.21%. Comparing base (607512e) to head (974c515).
⚠️ Report is 7 commits behind head on main.

@@            Coverage Diff             @@
##             main     #599      +/-   ##
==========================================
- Coverage   79.36%   79.21%   -0.16%     
==========================================
  Files         106      107       +1     
  Lines       25979    26050      +71     
  Branches    25979    26050      +71     
==========================================
+ Hits        20619    20635      +16     
- Misses       3073     3127      +54     
- Partials     2287     2288       +1     

see 3 files with indirect coverage changes

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

Copy link
Collaborator

@AlexD10S AlexD10S left a comment

Choose a reason for hiding this comment

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

Thanks for catching this and fixing it!
The unit test for the v6! contracts is currently failing, but it’s unrelated to this PR. It will be addressed in #582

@AlexD10S AlexD10S changed the title Remove sp-weights from the list of dependencies chore: remove sp-weights from the list of dependencies Sep 1, 2025
@AlexD10S AlexD10S merged commit 8c84fc0 into r0gue-io:main Sep 1, 2025
18 of 21 checks passed
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.

2 participants