Skip to content

fix: add missing callback res after refactor#1560

Merged
glevco merged 1 commit intomasterfrom
fix/callback-res
Jan 21, 2026
Merged

fix: add missing callback res after refactor#1560
glevco merged 1 commit intomasterfrom
fix/callback-res

Conversation

@glevco
Copy link
Contributor

@glevco glevco commented Jan 20, 2026

Motivation

There's a bug introduced by #1542.

Acceptance Criteria

  • Add missing param to deferred callback.
  • Add test covering the issue.

Checklist

  • If you are requesting a merge into master, confirm this code is production-ready and can be included in future releases as soon as it gets merged

@glevco glevco self-assigned this Jan 20, 2026
@glevco glevco moved this from Todo to In Progress (Done) in Hathor Network Jan 20, 2026
@github-actions
Copy link

github-actions bot commented Jan 20, 2026

🐰 Bencher Report

Branchfix/callback-res
Testbedubuntu-22.04
Click to view all benchmark results
BenchmarkLatencyBenchmark Result
minutes (m)
(Result Δ%)
Lower Boundary
minutes (m)
(Limit %)
Upper Boundary
minutes (m)
(Limit %)
sync-v2 (up to 20000 blocks)📈 view plot
🚷 view threshold
1.68 m
(-2.28%)Baseline: 1.72 m
1.55 m
(92.10%)
2.06 m
(81.43%)
🐰 View full continuous benchmarking report in Bencher

@glevco glevco moved this from In Progress (Done) to In Progress (WIP) in Hathor Network Jan 20, 2026
luislhl
luislhl previously approved these changes Jan 20, 2026
msbrogli
msbrogli previously approved these changes Jan 20, 2026
@github-project-automation github-project-automation bot moved this from In Progress (WIP) to In Review (WIP) in Hathor Network Jan 20, 2026
@glevco glevco moved this from In Review (WIP) to In Progress (WIP) in Hathor Network Jan 20, 2026
@glevco glevco dismissed stale reviews from msbrogli and luislhl via bad586c January 21, 2026 00:53
@glevco glevco moved this from In Progress (WIP) to In Progress (Done) in Hathor Network Jan 21, 2026
@glevco glevco moved this from In Progress (Done) to In Review (WIP) in Hathor Network Jan 21, 2026
@codecov
Copy link

codecov bot commented Jan 21, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.29%. Comparing base (dae8929) to head (22ca0be).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1560      +/-   ##
==========================================
- Coverage   86.34%   86.29%   -0.05%     
==========================================
  Files         437      437              
  Lines       33610    33610              
  Branches     5248     5248              
==========================================
- Hits        29020    29005      -15     
- Misses       3588     3599      +11     
- Partials     1002     1006       +4     

☔ 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.

@glevco glevco merged commit e8640de into master Jan 21, 2026
17 of 18 checks passed
@glevco glevco deleted the fix/callback-res branch January 21, 2026 16:39
@github-project-automation github-project-automation bot moved this from In Review (WIP) to Waiting to be deployed in Hathor Network Jan 21, 2026
@jansegre jansegre mentioned this pull request Jan 21, 2026
2 tasks
@jansegre jansegre moved this from Waiting to be deployed to Done in Hathor Network Jan 21, 2026
@jansegre jansegre mentioned this pull request Jan 21, 2026
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants