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

feat:#4759 venus-market : add docker feature #131

Merged
merged 13 commits into from
Jun 9, 2022
Merged

feat:#4759 venus-market : add docker feature #131

merged 13 commits into from
Jun 9, 2022

Conversation

LinZexiao
Copy link
Collaborator

@LinZexiao LinZexiao commented May 13, 2022

Related Issues

filecoin-project/venus#4759

Proposed Changes

add docker features
add github action for automaticly pushing to dockerhub

Additional Info

Checklist
Before you mark the PR ready for review, please make sure that:

  • All commits have a clear commit message.
  • The PR title is in the form of of : : <#issue number> :
  • This PR has tests for new functionality or change in behaviour
  • If new user-facing features are introduced, clear usage guidelines and / or documentation updates should be included in venus docs or Discussion Tutorials.
  • CI is green

@codecov-commenter
Copy link

codecov-commenter commented Jun 8, 2022

Codecov Report

Merging #131 (0e1055e) into main (25483cb) will decrease coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #131      +/-   ##
==========================================
- Coverage   18.78%   18.76%   -0.03%     
==========================================
  Files         169      169              
  Lines       17738    17738              
==========================================
- Hits         3332     3328       -4     
- Misses      13972    13975       +3     
- Partials      434      435       +1     
Flag Coverage Δ
unittests 18.76% <ø> (-0.03%) ⬇️

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

Impacted Files Coverage Δ
blockstore/badger/blockstore.go 55.90% <0.00%> (-0.46%) ⬇️
paychmgr/simple.go 78.48% <0.00%> (-0.23%) ⬇️

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 c117e1c into main Jun 9, 2022
@diwufeiwen diwufeiwen deleted the feat/docker branch June 9, 2022 09:12
litai686 pushed a commit to litai686/venus-market that referenced this pull request Jul 30, 2022
…/feat/dtynn/worker_management

Feat/dtynn/worker management
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.

3 participants