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

Update CSPV naming #276

Merged
merged 1 commit into from
Jun 24, 2024
Merged

Update CSPV naming #276

merged 1 commit into from
Jun 24, 2024

Conversation

aarshkshah1992
Copy link
Contributor

This reflects the changes made in filecoin-project/builtin-actors#1553

rvagg
rvagg reviewed Jun 21, 2024
View reviewed changes
Copy link
Member

@rvagg rvagg left a comment

Choose a reason for hiding this comment

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

This seems fine. I'm out of the loop on naming so I'll take your word for it. But I believe this needs to be against feat/nv23 instead of master.

@aarshkshah1992 aarshkshah1992 changed the base branch from master to feat/nv23 June 21, 2024 09:34
@aarshkshah1992 aarshkshah1992 changed the base branch from feat/nv23 to master June 21, 2024 10:22
@aarshkshah1992 aarshkshah1992 mentioned this pull request Jun 21, 2024
Merged
8 tasks
@aarshkshah1992 aarshkshah1992 merged commit cf50ca0 into master Jun 24, 2024
5 checks passed
@aarshkshah1992 aarshkshah1992 deleted the fix/update-cspv-params branch June 24, 2024 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rvagg rvagg rvagg approved these changes

@ZenGround0 ZenGround0 ZenGround0 approved these changes

Assignees

@aarshkshah1992 aarshkshah1992

Labels
None yet
Projects
nv23 Track Board
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@aarshkshah1992 @rvagg @ZenGround0