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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs: fix bentoml containerize command help message #2957
Conversation
Signed-off-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com>
Signed-off-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com>
Codecov Report
@@ Coverage Diff @@
## main #2957 +/- ##
==========================================
- Coverage 70.60% 69.71% -0.90%
==========================================
Files 104 104
Lines 9538 9538
==========================================
- Hits 6734 6649 -85
- Misses 2804 2889 +85
|
Signed-off-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com>
bentoml/bentos.py
Outdated
f'To run your newly built Bento container, use: "docker run -it --rm -p 3000:3000 {docker_image_tag}"' | ||
'Successfully built docker image for "%s" with tags "%s"', | ||
str(bento.tag), | ||
docker_image_tag if not custom_tags else ",".join(docker_image_tag), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems to me like we should make this variable always an array.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I remember we have this discussion before, let me do that.
Co-authored-by: Sauyon Lee <2347889+sauyon@users.noreply.github.com>
Signed-off-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
馃憤
Er, these test failures look new, do you know what's going on? |
no clue, from 37beaf8 out of nowhere. |
Co-authored-by: Sauyon Lee <2347889+sauyon@users.noreply.github.com>
update docstring to address #2956
This PR also fix a tiny messaging format when user did custom tag. I forgot to push the change for the message fixing multiple tags 馃槃