awslabs / amazon-kinesis-video-streams-pic

Apache License 2.0
49 stars 51 forks source link

Remove an extra comma that made the format incorrect #216

Closed daveisfera closed 1 year ago

daveisfera commented 1 year ago

Log was incorrect because of an extra , that prevented string concatenation

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

codecov-commenter commented 1 year ago

Codecov Report

Patch coverage: 50.00% and project coverage change: +0.02% :tada:

Comparison is base (87f42b6) 75.20% compared to head (dc915fa) 75.23%. Report is 4 commits behind head on master.

:exclamation: Current head dc915fa differs from pull request most recent head 701c839. Consider uploading reports for the commit 701c839 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #216 +/- ## ========================================== + Coverage 75.20% 75.23% +0.02% ========================================== Files 52 52 Lines 10182 10178 -4 ========================================== Hits 7657 7657 + Misses 2525 2521 -4 ``` | [Files Changed](https://app.codecov.io/gh/awslabs/amazon-kinesis-video-streams-pic/pull/216?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=awslabs) | Coverage Δ | | |---|---|---| | [src/client/src/Stream.c](https://app.codecov.io/gh/awslabs/amazon-kinesis-video-streams-pic/pull/216?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=awslabs#diff-c3JjL2NsaWVudC9zcmMvU3RyZWFtLmM=) | `69.85% <50.00%> (ø)` | | ... and [1 file with indirect coverage changes](https://app.codecov.io/gh/awslabs/amazon-kinesis-video-streams-pic/pull/216/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=awslabs)

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

daveisfera commented 1 year ago

Done

disa6302 commented 1 year ago

Thank you!