Skip to content

refactor(collator): shuffle#1058

Closed
Mododo wants to merge 17 commits into
feature/slasher-sessionfrom
fvjknfvkjnfvdkjndfvkjndfvkjnvsdjkn
Closed

refactor(collator): shuffle#1058
Mododo wants to merge 17 commits into
feature/slasher-sessionfrom
fvjknfvkjnfvdkjndfvkjndfvkjnvsdjkn

Conversation

@Mododo
Copy link
Copy Markdown
Member

@Mododo Mododo commented Mar 25, 2026

No description provided.

@Mododo Mododo requested a review from MrWad3r March 25, 2026 04:11
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 25, 2026

🧪 Network Tests

To run network tests for this PR, use:

gh workflow run network-tests.yml -f pr_number=1058

Available test options:

  • Run all tests: gh workflow run network-tests.yml -f pr_number=1058
  • Run specific test: gh workflow run network-tests.yml -f pr_number=1058 -f test_selection=ping-pong

Test types: destroyable, ping-pong, one-to-many-internal-messages, fq-deploy, nft-index, persistent-sync

Results will be posted as workflow runs in the Actions tab.

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 25, 2026

Codecov Report

❌ Patch coverage is 29.66776% with 1503 lines in your changes missing coverage. Please review.
✅ Project coverage is 56.18%. Comparing base (bff6339) to head (c778483).
⚠️ Report is 42 commits behind head on feature/slasher-session.

Files with missing lines Patch % Lines
slasher/src/storage/mod.rs 0.00% 307 Missing ⚠️
slasher/src/lib.rs 0.00% 256 Missing ⚠️
slasher/src/storage/models.rs 0.00% 252 Missing ⚠️
slasher/src/collector/validator_events.rs 0.00% 180 Missing ⚠️
slasher/src/bc/stub_contract.rs 33.13% 110 Missing and 3 partials ⚠️
slasher/src/analyzer.rs 0.00% 99 Missing ⚠️
slasher/src/bc/mod.rs 31.31% 68 Missing ⚠️
cli/src/cmd/tools/gen_zerostate.rs 0.00% 52 Missing ⚠️
slasher/src/util.rs 79.90% 36 Missing and 6 partials ⚠️
slasher-traits/src/validator.rs 73.85% 39 Missing and 1 partial ⚠️
... and 12 more
Additional details and impacted files
@@                     Coverage Diff                     @@
##           feature/slasher-session    #1058      +/-   ##
===========================================================
- Coverage                    56.92%   56.18%   -0.74%     
===========================================================
  Files                          431      442      +11     
  Lines                        72618    74177    +1559     
  Branches                     72618    74177    +1559     
===========================================================
+ Hits                         41335    41674     +339     
- Misses                       29285    30504    +1219     
- Partials                      1998     1999       +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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Mododo Mododo force-pushed the fvjknfvkjnfvdkjndfvkjndfvkjnvsdjkn branch 2 times, most recently from 9bfe0bf to ab76e7a Compare March 25, 2026 11:33
@Mododo Mododo changed the title fix(collator): nit fix(collator): shuffle Mar 25, 2026
@Mododo Mododo requested review from SmaGMan March 25, 2026 11:36
@Mododo Mododo force-pushed the fvjknfvkjnfvdkjndfvkjndfvkjnvsdjkn branch from ab76e7a to c778483 Compare April 1, 2026 06:13
@Mododo Mododo marked this pull request as draft April 1, 2026 16:04
@Mododo Mododo changed the title fix(collator): shuffle refactor(collator): shuffle Apr 1, 2026
@MrWad3r MrWad3r force-pushed the feature/slasher-session branch from e626422 to f86db80 Compare April 9, 2026 10:54
@MrWad3r MrWad3r force-pushed the feature/slasher-session branch from f86db80 to 8542d1f Compare April 9, 2026 12:46
@Mododo Mododo closed this Apr 13, 2026
@Mododo Mododo deleted the fvjknfvkjnfvdkjndfvkjndfvkjnvsdjkn branch April 13, 2026 07:38
@Mododo
Copy link
Copy Markdown
Member Author

Mododo commented Apr 13, 2026

moved to batch PR #1067

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.

3 participants