Skip to content

feat: add stake manager#597

Merged
aloknerurkar merged 13 commits intomainfrom
notifications.1
May 7, 2025
Merged

feat: add stake manager#597
aloknerurkar merged 13 commits intomainfrom
notifications.1

Conversation

@aloknerurkar
Copy link
Copy Markdown
Collaborator

@aloknerurkar aloknerurkar commented Feb 11, 2025

Describe your changes

Addresses #549

Fixes #549

Checklist before requesting a review

  • I have added tests that prove my fix is effective or that my feature works
  • I have made corresponding changes to the documentation

Comment thread p2p/pkg/stakemanager/stakemanager.go Outdated
Comment thread p2p/pkg/stakemanager/stakemanager.go
Copy link
Copy Markdown
Contributor

@mrekucci mrekucci left a comment

Choose a reason for hiding this comment

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

Aside from the comments, LGTM!

Comment thread p2p/pkg/stakemanager/stakemanager.go
Comment thread p2p/pkg/stakemanager/stakemanager.go Outdated
@aloknerurkar aloknerurkar force-pushed the notifications.1 branch 2 times, most recently from 372df46 to 993c55e Compare May 6, 2025 14:26
@aloknerurkar aloknerurkar merged commit 24df4e8 into main May 7, 2025
5 checks passed
@aloknerurkar aloknerurkar deleted the notifications.1 branch May 7, 2025 13:42
aloknerurkar added a commit that referenced this pull request Aug 5, 2025
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.

Implement StakeManager

3 participants