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

chore: update fvm #5605

Merged
merged 4 commits into from
Dec 22, 2022
Merged

chore: update fvm #5605

merged 4 commits into from
Dec 22, 2022

Conversation

simlecode
Copy link
Collaborator

@simlecode simlecode commented Dec 21, 2022

关联的Issues (Related Issues)

#5550
filecoin-project/lotus#9915
filecoin-project/lotus#9916

改动 (Proposed Changes)

  • update ffi
  • generate builtin actor code
  • reduce ResolveToKeyAddr implementation

附注 (Additional Info)

自查清单 (Checklist)

在你认为本 PR 满足被审阅的标准之前,需要确保 / Before you mark the PR ready for review, please make sure that:

  • 符合Venus项目管理规范中关于PR的相关标准 / The PR follows the PR standards set out in the Venus project management guidelines
  • 具有清晰明确的commit message / All commits have a clear commit message.
  • 包含相关的的测试用例或者不需要新增测试用例 / This PR has tests for new functionality or change in behaviour or not need to add new tests.
  • 存在兼容性问题(接口, 配置,数据,灰度),如果存在需要进行文档说明 / This PR has compatibility issues (API, Configuration, Data, GrayRelease), if so, need to be documented.
  • 包含相关的的指南以及文档或者不需要新增文档 / This PR has updated usage guidelines and documentation or not need
  • 通过必要的检查项 / All checks are green

@simlecode simlecode requested a review from diwufeiwen December 21, 2022 10:03
@simlecode simlecode self-assigned this Dec 21, 2022
@simlecode simlecode added the nv18 nv18-fevm label Dec 21, 2022
Copy link
Contributor

@diwufeiwen diwufeiwen left a comment

Choose a reason for hiding this comment

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

Lgtm

@diwufeiwen diwufeiwen merged commit 6e79de6 into feat/nv18-fevm Dec 22, 2022
@diwufeiwen diwufeiwen deleted the chore/update-fvm branch December 22, 2022 01:36
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
nv18 nv18-fevm
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants