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 4 (grpc query) #1117

Merged
merged 18 commits into from
Mar 4, 2024
Merged

stakeibc file re-org part 4 (grpc query) #1117

merged 18 commits into from
Mar 4, 2024

Conversation

sampocs
Copy link
Collaborator

@sampocs sampocs commented Mar 2, 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

Consolidates grpc queries into one file

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
This one was far easier because of the structure for what its worth

@sampocs sampocs merged commit c991249 into main Mar 4, 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