-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
docs(actors): builtin-actors development docs #12551
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Co-authored-by: Steve Loeppky <[email protected]>
9f02fa4
to
c3b104c
Compare
This is ready for review with the first section of builtin-actors -> lotus. But it also hopefully has enough foundational stuff that adding the additional sections for go-state-types, migrations and ref-fvm are just additive and not repetitive and tedious. I think we land this now and the remaining sections can be done over time. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I looked at the latest changes - looks good to me - nice work!
Co-authored-by: Steve Loeppky <[email protected]>
Ref: filecoin-project/builtin-actors#1567
@rjan90 @BigLep I started banging this out as I was closing out 0081 and preparing a fresh branch based on these instructions for @snissn to get him started on MCOPY. I'll finish off the "Builtin-actors-only changes" section tomorrow, along with the MCOPY starter branch, unless something else more urgent comes up. The other sections can remain TODO until later, although the state migration one probably isn't too hard, now that I've done it for 0081; but it can be done separately.