build(deps): Bump github.com/vladimirvivien/go4vl from 0.0.2 to 0.0.5

edgexfoundry / device-usb-camera

Owner: Device WG

Apache License 2.0

12 stars 28 forks source link

build(deps): Bump github.com/vladimirvivien/go4vl from 0.0.2 to 0.0.5 #214

Closed jinlinGuan closed 1 year ago

jinlinGuan commented 1 year ago

update go4vl format constants

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-go/blob/main/.github/Contributing.md

PR Checklist

Please check if your PR fulfills the following requirements:

[x] I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
[x] I am not introducing a new dependency (add notes below if you are)
[ ] I have added unit tests for the new feature or bug fix (if not, why?)
[ ] I have fully tested (add details below) this the new feature or bug fix (if not, why?)
[ ] I have opened a PR for the related docs change (if not, why?)

Testing Instructions

New Dependency Instructions (If applicable)

codecov-commenter commented 1 year ago

Codecov Report

Merging #214 (171ca04) into main (6092c90) will not change coverage. The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@          Coverage Diff          @@
##            main    #214   +/-   ##
=====================================
  Coverage   3.05%   3.05%           
=====================================
  Files          6       6           
  Lines        687     687           
=====================================
  Hits          21      21           
  Misses       666     666

Impacted Files	Coverage Δ
internal/driver/ffmpeg.go	`16.50% <100.00%> (ø)`

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

lenny-goodell commented 1 year ago

@cloudxxx8 , FYI, we had been holding back on the newer versions of this module due to issues. @ajcasagrande thoughts on using this version?

cloudxxx8 commented 1 year ago

sorry, I didn't realize it. I thought it was hold due to the CI validation error, and Ginny just solved it. the upgrade of this module resolved some bugs, especially about the memory leak. may we list the problem we encountered and we can open issues to the module and decide whether we want to revert this PR?