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

ci: setup cppcheck for codeclimate #159

Merged
merged 1 commit into from
Oct 11, 2019
Merged

ci: setup cppcheck for codeclimate #159

merged 1 commit into from
Oct 11, 2019

Conversation

faustbrian
Copy link
Contributor

@faustbrian faustbrian commented Oct 11, 2019

Summary

C++ isn't supported out of the box but requires a plugin to be configured. See https://docs.codeclimate.com/docs/cppcheck and https://github.com/antiagainst/codeclimate-cppcheck#configuration.

Checklist

  • Documentation (if necessary)
  • Tests (if necessary)
  • Ready to be merged

@ghost ghost added the Complexity: Low Less than 64 lines changed. label Oct 11, 2019
@codecov
Copy link

codecov bot commented Oct 11, 2019

Codecov Report

Merging #159 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #159   +/-   ##
=======================================
  Coverage   91.59%   91.59%           
=======================================
  Files          33       33           
  Lines         857      857           
=======================================
  Hits          785      785           
  Misses         72       72

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e888899...458b052. Read the comment docs.

@faustbrian faustbrian merged commit e34ab67 into master Oct 11, 2019
@ghost ghost deleted the faustbrian-patch-1 branch October 11, 2019 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Complexity: Low Less than 64 lines changed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant