Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

stakeibc file re-org part 2 (epochly functions) #1113

Merged
merged 11 commits into from
Mar 1, 2024
Merged

Conversation

sampocs
Copy link
Collaborator

@sampocs sampocs commented Mar 1, 2024

Context

Stakeibc has grown quite large over the years. This PR does a file re-org cleanup to better organize functions and remove ignite tech debt. This should make it easier to onboard new devs to the codebase.

Note to reviewer: There were no logical changes made in this PR. It only copy pasted functions to different files. When reviewing, I'd recommend scrolling each commit and making sure all functions are accounted for during the move.

Brief Changelog

Logically grouped epochly code into the following files:

  • transfer.go
  • delegation.go
  • unbonding.go
  • redemption_sweep.go
  • claim.go

Copy link
Contributor

@ethan-stride ethan-stride left a comment

Choose a reason for hiding this comment

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

LGTM
Same as the previous one, I just verified all functions which were deleted still exist as being moved to a different place.

@sampocs sampocs merged commit 731e28b into main Mar 1, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants