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

fix: setuptools doesn't include bentoml_cli #2898

Merged
merged 4 commits into from
Aug 12, 2022
Merged

Conversation

bojiang
Copy link
Member

@bojiang bojiang commented Aug 12, 2022

No description provided.

@bojiang bojiang requested a review from a team as a code owner August 12, 2022 09:38
@bojiang bojiang requested review from parano and removed request for a team August 12, 2022 09:38
aarnphm
aarnphm previously approved these changes Aug 12, 2022
setup.cfg Outdated Show resolved Hide resolved
setup.cfg Outdated Show resolved Hide resolved
setup.cfg Outdated Show resolved Hide resolved
@bojiang bojiang requested a review from aarnphm August 12, 2022 09:40
@codecov
Copy link

codecov bot commented Aug 12, 2022

Codecov Report

Merging #2898 (21bdea6) into main (5860906) will increase coverage by 0.05%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2898      +/-   ##
==========================================
+ Coverage   70.17%   70.23%   +0.05%     
==========================================
  Files         120      120              
  Lines        9792     9792              
==========================================
+ Hits         6872     6877       +5     
+ Misses       2920     2915       -5     
Impacted Files Coverage Δ
bentoml/_internal/resource.py 76.96% <0.00%> (+2.42%) ⬆️
bentoml/_internal/utils/uri.py 89.47% <0.00%> (+5.26%) ⬆️

@aarnphm aarnphm changed the title fix: setup.cfg fix: setuptools doesn't include bentoml_cli Aug 12, 2022
@aarnphm aarnphm merged commit 29ae65a into bentoml:main Aug 12, 2022
@bojiang bojiang deleted the fix-setup branch August 17, 2022 03:10
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.

None yet

2 participants