Closed Thaodan closed 1 month ago
Hello @Thaodan! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:
Attention: Patch coverage is 33.33333%
with 2 lines
in your changes missing coverage. Please review.
Project coverage is 36.29%. Comparing base (
f35895d
) to head (66b93be
). Report is 51 commits behind head on master.
Files with missing lines | Patch % | Lines |
---|---|---|
osc/build.py | 0.00% | 2 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Thanks for the patch. LGTM, merging.
Add --no-timestamps parameter which already exists in build. Help text copied from build. Include parameters for the same parameter in buildlog for easier use.