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.

Sign up for GitHub

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

Jump to bottom

fix: fix incorrect MinerGetBaseInfo #4617

Merged
merged 1 commit into from
Aug 6, 2024
Merged

fix: fix incorrect MinerGetBaseInfo #4617

merged 1 commit into from
Aug 6, 2024

Conversation

LesnyRumcajs
Copy link
Member

@LesnyRumcajs LesnyRumcajs commented Aug 5, 2024 •
edited
Loading

Summary of changes

Changes introduced in this pull request:

  • fixes the Filecoin.MinerGetBaseInfo errors.

Reference issue to close (if applicable)

Closes #4610

Other information and links

Change checklist

  • I have performed a self-review of my own code,
  • I have made corresponding changes to the documentation,
  • I have added tests that prove my fix is effective or that my feature works (if possible),
  • I have made sure the CHANGELOG is up-to-date. All user-facing changes should be reflected in this document.

@LesnyRumcajs LesnyRumcajs force-pushed the fix-miner-get-base-info branch 3 times, most recently from 6a282ca to 8941a7d Compare August 5, 2024 10:52
@LesnyRumcajs LesnyRumcajs force-pushed the fix-miner-get-base-info branch from 8941a7d to 0a85621 Compare August 6, 2024 09:36
@LesnyRumcajs LesnyRumcajs marked this pull request as ready for review August 6, 2024 09:55
@LesnyRumcajs LesnyRumcajs requested a review from a team as a code owner August 6, 2024 09:55
@LesnyRumcajs LesnyRumcajs requested review from lemmih and hanabi1224 and removed request for a team August 6, 2024 09:55
@LesnyRumcajs LesnyRumcajs added this pull request to the merge queue Aug 6, 2024
Merged via the queue into main with commit d4408ae Aug 6, 2024
52 checks passed
@LesnyRumcajs LesnyRumcajs deleted the fix-miner-get-base-info branch August 6, 2024 13:18
@LesnyRumcajs LesnyRumcajs mentioned this pull request Aug 7, 2024
Closed
LesnyRumcajs added a commit that referenced this pull request Aug 7, 2024
@LesnyRumcajs
Revert "fix: fix incorrect MinerGetBaseInfo (#4617)"
5425992 
This reverts commit d4408ae.
@LesnyRumcajs LesnyRumcajs mentioned this pull request Aug 7, 2024
Merged
4 tasks
github-merge-queue bot pushed a commit that referenced this pull request Aug 7, 2024
@LesnyRumcajs
Revert "fix: fix incorrect MinerGetBaseInfo (#4617)" (#4627)
b8d5f3b
LesnyRumcajs added a commit that referenced this pull request Aug 8, 2024
@LesnyRumcajs
Reapply "fix: fix incorrect MinerGetBaseInfo (#4617)" (#4627)
d8d32ed 
This reverts commit b8d5f3b.
LesnyRumcajs added a commit that referenced this pull request Aug 8, 2024
@LesnyRumcajs
Reapply "fix: fix incorrect MinerGetBaseInfo (#4617)" (#4627)
996e29e 
This reverts commit b8d5f3b.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hanabi1224 hanabi1224 hanabi1224 approved these changes

@lemmih lemmih lemmih approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Fix Filecoin.MinerGetBaseInfo
3 participants
@LesnyRumcajs @lemmih @hanabi1224