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

Add a shed util to count miners by post type #6169

Merged
merged 1 commit into from
May 10, 2021
Merged

Add a shed util to count miners by post type #6169

merged 1 commit into from
May 10, 2021

Conversation

arajasek
Copy link
Contributor

@arajasek arajasek commented May 3, 2021

No description provided.

@arajasek arajasek requested a review from magik6k as a code owner May 3, 2021 00:29
@arajasek arajasek requested a review from ZenGround0 May 3, 2021 23:17
ZenGround0
ZenGround0 approved these changes May 4, 2021
View reviewed changes
Copy link
Contributor

@ZenGround0 ZenGround0 left a comment

Choose a reason for hiding this comment

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

General observation: we could group together utils that want direct datastore access for speed. Maybe a lotus-basement to go with the lotus-shed? The active cron util I just wrote should do it this way instead.

@arajasek arajasek merged commit 9977cb2 into master May 10, 2021
@arajasek arajasek deleted the asr/shed-minertypes branch May 10, 2021 23:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ZenGround0 ZenGround0 ZenGround0 approved these changes

@magik6k magik6k Awaiting requested review from magik6k

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

Successfully merging this pull request may close these issues.
2 participants
@arajasek @ZenGround0