Skip to content

Added support for Black Duck Coverity (formerly Synopsys) for newer versions #189

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Alaeddin03
Copy link

Added support for Black Duck Coverity (formerly Synopsys) for newer versions (version 10 as of the date of this PR).

Newer versions greater than 3 were handled in the "else" block even though they can be safely parsed by the same configuration for version 3.

Newer versions greater than 3 are getting handled in the "else" block even though they can be safely parsed by the same configuration for version 3 (as of version 10 currently)
@davewichers
Copy link
Contributor

@darkspirit510 - Can you check this out and let me know if you want any changes, or if it's OK to merge? Thx

@darkspirit510
Copy link
Contributor

@Alaeddin03 thank you for the PR. Did Black Duck change anything but the version? If not, could you either add a test or provide an actual result file to me ([email protected]) so I can create one?

@Alaeddin03
Copy link
Author

@darkspirit510 - Yes the only change was the version. I added a test for it.

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

Successfully merging this pull request may close these issues.

3 participants