Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

remove support for nv14 and actors v6. #556

Merged
merged 1 commit into from
May 13, 2022
Merged

Conversation

raulk
Copy link
Member

@raulk raulk commented May 13, 2022

When this repo was started, the latest network version on mainnet was nv14. In order to run canaries against mainnet (M0 and M0.5 milestones), we added support for nv14. Now that mainnet is on nv15 and actors v7, we can remove support for nv14. Reasons:

  • We don't intend to maintain actors v6 under builtin-actors.
  • More specifically, actors v6 won't be updated to absorb the syscall changes being introduced in merge syscall changes to master #533.
  • We can shed unneeded code by making the deprecation official.

@raulk raulk marked this pull request as ready for review May 13, 2022 15:49
@raulk raulk requested review from Stebalien and arajasek May 13, 2022 15:49
@Stebalien Stebalien merged commit e5e147a into master May 13, 2022
@Stebalien Stebalien deleted the raulk/remove-nv14 branch May 13, 2022 15:50
@Stebalien
Copy link
Member

Merging before anyone has a chance to tell me why I shouldn't... ❤️.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants