search

jazzband / django-silk

Silky smooth profiling for Django
MIT License
4.42k stars 333 forks source link

Clarify max body size units and format readme #695

Closed AbhiMisRaw closed 3 months ago

AbhiMisRaw commented 9 months ago

It's my first contribution, as @SebCorbin said, It's an docs issue, so I've updated it BODY -> BYTE

Fixes #300

codecov[bot] commented 8 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (23ff43b) 86.65% compared to head (c43a471) 86.65%. Report is 4 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #695 +/- ## ======================================= Coverage 86.65% 86.65% ======================================= Files 52 52 Lines 2113 2113 ======================================= Hits 1831 1831 Misses 282 282 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

albertyw commented 3 months ago

The issue with BODY/BYTES was fixed in d40bb9dc. The formatting changes are fine but are separate from what this PR was trying to fix.