Skip to content

Conversation

@meocong
Copy link
Contributor

@meocong meocong commented Aug 27, 2021

@lamhoangtung Can you review it for me, some app need request size to be higher than default 200MB, so I changed it into 1GB

@meocong meocong added the enhancement New feature or request label Aug 27, 2021
lamhoangtung
lamhoangtung previously approved these changes Aug 27, 2021
Copy link
Contributor

@lamhoangtung lamhoangtung left a comment

Choose a reason for hiding this comment

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

LGTM

@codecov
Copy link

codecov bot commented Aug 27, 2021

Codecov Report

Merging #46 (305506c) into master (d81f6b5) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #46   +/-   ##
=======================================
  Coverage   54.11%   54.11%           
=======================================
  Files          46       46           
  Lines        4038     4038           
=======================================
  Hits         2185     2185           
  Misses       1853     1853           
Impacted Files Coverage Δ
mlchain/__init__.py 90.47% <100.00%> (ø)
mlchain/server/flask_server.py 60.24% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d81f6b5...305506c. Read the comment docs.

Copy link
Contributor

@lamhoangtung lamhoangtung left a comment

Choose a reason for hiding this comment

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

LGTM

@lamhoangtung lamhoangtung merged commit 33fe470 into master Aug 27, 2021
@lamhoangtung lamhoangtung deleted the starllete branch August 27, 2021 05:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants