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.

Sign up for GitHub

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

Jump to bottom

Enable coverity #265

Merged
merged 4 commits into from
Nov 10, 2015
Merged

Enable coverity #265

merged 4 commits into from
Nov 10, 2015

Conversation

simonharrer
Copy link
Contributor

Why?

  • coverity does static analysis of the source code and makes the found warnings available via reports. In addition to its own checks it also performs findbugs evaluations.

To fix still

  • coveralls gradle plugin and coverity gradle plugin do not get along. Coverity causes coveralls to fail by using/loading the wrong class. :(

@koppor
Copy link
Member

koppor commented Oct 27, 2015

👍

The failed build is kt3k/coveralls-gradle-plugin#47 (as we use 2.4.0). (Outdated Issue found by google at first: kt3k/coveralls-gradle-plugin#27? Should we update the coveralls plugin to 2.4.0x?

The only thing, that we changed, is the inclusion of coverty 1.3.708. Does that version depend on httpmime-4.1 which seems to be broken?

simonharrer added a commit that referenced this pull request Nov 10, 2015
@simonharrer
Merge pull request #265 from JabRef/enable-coverity
275b1c9 
Enable coverity
@simonharrer simonharrer merged commit 275b1c9 into master Nov 10, 2015
@stefan-kolb stefan-kolb deleted the enable-coverity branch November 10, 2015 12:34
koppor pushed a commit that referenced this pull request Apr 1, 2023
Squashed 'buildres/csl/csl-locales/' changes from 47edde1be3..a7c6f63e25
02b68e5 
a7c6f63e25 correct license to match the SPDX license identifier. (#281)
d704bf80af Update locales-nl-NL.xml (#229)
5ffb73b05a Bump nokogiri from 1.13.9 to 1.13.10 (#280)
04be62eda6 Update locales-pt-BR.xml (#251)
b4db583787 Update locales-pt-BR.xml (#265)
b656b1b6f9 Fix date format for Basque (#274)
e7ec9bff94 Bump nokogiri from 1.13.4 to 1.13.9 (#272)
9125705f62 Update locales-nl-NL.xml (#279)
87445b0b65 Add composer.json (#161)
2919a84bff Fix page label in NO locales

git-subtree-dir: buildres/csl/csl-locales
git-subtree-split: a7c6f63e25323ac2f375943417d7f778f875f11c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@simonharrer @koppor