Merge ~rodrigo-zaiden/ubuntu-security-tools:fix-uaudit-coverity-report into ubuntu-security-tools:master
Status: | Merged |
---|---|
Merged at revision: | 2e52337a30bdd4a8e1562a2015a0f227a8101d43 |
Proposed branch: | ~rodrigo-zaiden/ubuntu-security-tools:fix-uaudit-coverity-report |
Merge into: | ubuntu-security-tools:master |
Diff against target: |
17 lines (+4/-0) 1 file modified
audits/uaudit (+4/-0) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Steve Beattie | Approve | ||
Alex Murray | Pending | ||
Review via email: mp+424800@code.launchpad.net |
Commit message
uaudit: add coverity defects in summary
Static analysis results summary in JIRA.md has the coverity
count number of defects missing, re-adding.
In case coverity is not used, modify the text to say that.
Description of the change
in current scenario, the result summary for coverity in JIRA.md is as:
## Static analysis results summary
- $coverity coverity defects
- 37 warnings from bandit
- 0 brakeman defects
I think this is because the details for coverity are not being added.
When coverity is not used, I'm suggesting to change the '$coverity' text to be 'Skip check for', resulting in "Skip check for coverity defects"
Maybe, and possible, there are better ways to make this change, if so, please let me know so I can update the merge.
Thanks again!
LGTM and works correctly in local testing, thanks!