Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix coverity scan #8388

Merged
merged 3 commits into from Mar 12, 2020
Merged

Fix coverity scan #8388

merged 3 commits into from Mar 12, 2020

Conversation

prologic
Copy link
Contributor

Summary

As per title

Component Name
  • area/ci
Test Plan
  • This PR itself

I originally got a CI failure for this, so the workflow works but it
was previously not tested orrectly, hence the addition of pull_request:
to the workflow to ensure it runs inside a pull request that touches either
the workflow or the script itself.

Additional Information

@squash-labs
Copy link

squash-labs bot commented Mar 12, 2020

Manage this branch in Squash

Test this branch here: https://fix-coverity-scan-2-f045c.squash.io

@prologic
Copy link
Contributor Author

Coverity Scan / coverity (pull_request) Successful in 3m

@prologic prologic merged commit a18ec2d into master Mar 12, 2020
@prologic prologic deleted the fix_coverity_scan_2 branch March 31, 2020 11:23
Saruspete pushed a commit to Saruspete/netdata that referenced this pull request May 21, 2020
* Fix coverity scan

* Add path filter to pull_request: condition

* Add missing dependency
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants