Skip to content

feat: expose market ProviderSectors access on state-types abstraction#12978

Merged
rvagg merged 2 commits intomasterfrom
rvagg/provider-sectors
Mar 26, 2025
Merged

feat: expose market ProviderSectors access on state-types abstraction#12978
rvagg merged 2 commits intomasterfrom
rvagg/provider-sectors

Conversation

@rvagg
Copy link
Copy Markdown
Member

@rvagg rvagg commented Mar 25, 2025

Closes: #11997

See the itest for how this is used.

Depends on filecoin-project/go-state-types#378

@rvagg rvagg requested a review from LexLuthr March 25, 2025 00:59
@github-project-automation github-project-automation Bot moved this to 📌 Triage in FilOz Mar 25, 2025
@rvagg rvagg force-pushed the rvagg/provider-sectors branch 3 times, most recently from f90cbac to cbe718a Compare March 25, 2025 01:10
@BigLep BigLep moved this from 📌 Triage to 🔎 Awaiting Review in FilOz Mar 25, 2025
@rvagg rvagg force-pushed the rvagg/provider-sectors branch from cbe718a to fabb5e5 Compare March 25, 2025 04:59
Copy link
Copy Markdown
Contributor

@LexLuthr LexLuthr left a comment

Choose a reason for hiding this comment

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

Looks good! Thank you for the example usage. This should be very handy.

@github-project-automation github-project-automation Bot moved this from 🔎 Awaiting Review to ✔️ Approved by reviewer in FilOz Mar 25, 2025
Copy link
Copy Markdown
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.

Nothing looking controversial here

@rvagg rvagg enabled auto-merge (squash) March 26, 2025 03:18
@rvagg rvagg merged commit 614868b into master Mar 26, 2025
91 checks passed
@rvagg rvagg deleted the rvagg/provider-sectors branch March 26, 2025 03:23
@github-project-automation github-project-automation Bot moved this from ✔️ Approved by reviewer to 🎉 Done in FilOz Mar 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🎉 Done

Development

Successfully merging this pull request may close these issues.

New method to access ProviderSectors HAMT from market actor

5 participants