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

Add initial support for Travis CI #5

Merged
merged 2 commits into from
Oct 5, 2017
Merged

Add initial support for Travis CI #5

merged 2 commits into from
Oct 5, 2017

Conversation

misery
Copy link
Contributor

@misery misery commented Sep 29, 2017

No description provided.

@misery misery merged commit a5b97b5 into Governikus:community Oct 5, 2017
@misery misery deleted the travis branch October 5, 2017 11:36
@GitMensch
Copy link

@misery: I highly suggest to register for coverity scan and automate the checks via Travis - see https://scan.coverity.com/github

@misery
Copy link
Contributor Author

misery commented Apr 12, 2018

@GitMensch
Thanks for the suggestion.
We already registered coverity scan: https://scan.coverity.com/projects/governikus-ausweisapp2

@GitMensch
Copy link

GitMensch commented Apr 12, 2018

@misery This is good. What do you think about adding the badge to the readme?
For example with

[![Coverity Scan Build Status](https://scan.coverity.com/projects/governikus-ausweisapp2/badge.svg)](https://scan.coverity.com/projects/governikus-ausweisapp2)

shows as Coverity Scan Build Status

@misery
Copy link
Contributor Author

misery commented Apr 13, 2018

@GitMensch
Well, I like those badges. But coverity scan is checked carefully and there are some false-positives like these "5 new defects". We do not want to scare people about security issues that do not exist.

@GitMensch
Copy link

@misery I see. But if you are investigating these in general and mark false-positives as false-positives + trigger recheck those should go away from the badge and you show to users that you actually do coverity check.

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