Skip to content

Remove status main loop from Pool#426

Merged
plebhash merged 3 commits intostratum-mining:mainfrom
Shourya742:2026-04-13-remove-status-main-loop
Apr 14, 2026
Merged

Remove status main loop from Pool#426
plebhash merged 3 commits intostratum-mining:mainfrom
Shourya742:2026-04-13-remove-status-main-loop

Conversation

@Shourya742
Copy link
Copy Markdown
Collaborator

Part of: #234, for now only migrates Pool

@Shourya742 Shourya742 force-pushed the 2026-04-13-remove-status-main-loop branch from faf7eba to 99faebd Compare April 13, 2026 07:34
Comment thread pool-apps/pool/src/lib/downstream/mod.rs
@Shourya742 Shourya742 force-pushed the 2026-04-13-remove-status-main-loop branch from 99faebd to 0e6476c Compare April 14, 2026 14:15
@plebhash
Copy link
Copy Markdown
Member

can we clean up commit history?

we're doing things (e.g.: f8d2243), regretting and then doing something else later (e.g.: 0e6476c)

@Shourya742 Shourya742 force-pushed the 2026-04-13-remove-status-main-loop branch from 0e6476c to 12f9f69 Compare April 14, 2026 15:35
@Shourya742
Copy link
Copy Markdown
Collaborator Author

can we clean up commit history?

we're doing things (e.g.: f8d2243), regretting and then doing something else later (e.g.: 0e6476c)

Done. Though I regret the match action as well ;>, couldn't come up with better atm.

Copy link
Copy Markdown
Member

@plebhash plebhash left a comment

Choose a reason for hiding this comment

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

tACK

@plebhash plebhash merged commit 0c01d22 into stratum-mining:main Apr 14, 2026
12 of 13 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