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.

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

Prep/v2.2.0 rc2 #149

Merged
merged 16 commits into from
Jun 25, 2022
Merged

Prep/v2.2.0 rc2 #149

merged 16 commits into from
Jun 25, 2022

Conversation

simlecode
Copy link
Collaborator

No description provided.

simlecode and others added 12 commits June 6, 2022 16:07
* refactor load v8 actors before solo/pool execution
* fix crush caused by creation a 0 interval timer
* chore: upgrade venus & gateway

* chore: upgrade venus
* fix placeholder in output log format string

* feat: top level index local persistence (#140)

* top level index local persistence

* fix typo

* add: dockerfile & docker dir

* fix: docker push action rely on err branch

* add: docker-compose

* add: wait-for-it script

* add: more readable log

* feat: config snippet for compose.sh

* feat: add publish docker image action

* feat:add default piece storage path

* fix: change action trigger to making tag

* chore: ignore .vscode

* chore: rm the test workflow of docker

* style: add blank line to the end of the file

* fix: adjust filecoin-ffi subrepo version

* fix: TestPaychGetRestartAfterAddFundsMsg may stuck in forever waiting (#141)

* fix TestPaychGetRestartAfterAddFundsMsg may stuck in forever waiting

* fix : retrieval market: retrieval provider begins data transfer  in incorrect state. (#146)

* fix retrieval provider transfer data without waiting payment

Co-authored-by: tom <69969590+simlecode@users.noreply.github.com>
Co-authored-by: tanlang <lin.zexiao@qq.com>
Co-authored-by: Tiance <40375298+diwufeiwen@users.noreply.github.com>
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

@simlecode simlecode requested a review from zl03jsj June 16, 2022 05:05
* fix creation of incorrect signer type

* refactor minor refactor of prepare

* fix remove unreachable code
@codecov-commenter
Copy link

codecov-commenter commented Jun 17, 2022

Codecov Report

Merging #149 (85590cc) into main (1be8676) will decrease coverage by 0.02%.
The diff coverage is 8.20%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #149      +/-   ##
==========================================
- Coverage   18.83%   18.81%   -0.03%     
==========================================
  Files         169      170       +1     
  Lines       17755    17802      +47     
==========================================
+ Hits         3345     3349       +4     
- Misses      13977    14021      +44     
+ Partials      433      432       -1     
Flag Coverage Δ
unittests 18.81% <8.20%> (-0.03%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
api/clients/modules.go 0.00% <0.00%> (ø)
api/clients/sign_client.go 0.00% <0.00%> (ø)
api/clients/venus_message_client.go 0.00% <0.00%> (ø)
api/impl/venus_market.go 0.00% <0.00%> (ø)
cli/actor.go 0.00% <0.00%> (ø)
client/client.go 9.20% <0.00%> (-0.04%) ⬇️
cmd/load_builtin_actor.go 0.00% <0.00%> (ø)
cmd/market-client/main.go 0.00% <0.00%> (ø)
cmd/venus-market/main.go 0.00% <0.00%> (ø)
cmd/venus-market/pool-run.go 0.00% <0.00%> (ø)
... and 28 more

@zl03jsj
Copy link
Contributor

zl03jsj commented Jun 24, 2022

@simlecode
由于main分支上已经合并了不少pr, 此分支需要先将main上的修改合并过来.

simlecode and others added 3 commits June 24, 2022 16:55
* chore: upgrade venus & gateway & messager

* chore: upgrade ffi
* fix: if a owner is multi-sig address (#151)

* fix if a owner is multi-sig address

Co-authored-by: hunjixin <1084400399@qq.com>

* fix:allow storage miner actor (#152)

Co-authored-by: zl <zl03jsj@163.com>
Co-authored-by: hunjixin <1084400399@qq.com>
Co-authored-by: Mike <41407352+hunjixin@users.noreply.github.com>
* fix: fix load builtin actors

* build: bump version to v2.2.0-rc2
Copy link
Contributor

@zl03jsj zl03jsj left a comment

Choose a reason for hiding this comment

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

看起来没啥问题, 有一个疑问是需要确认一下的已经添加评论.
鉴于本次提交内容较多, 提交的又是主分支, 是否需要先跑一下再合并?

@zl03jsj zl03jsj merged commit 945a84d into main Jun 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants