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 missing overrides #478

Merged
merged 1 commit into from Sep 28, 2019

Conversation

@amarsman
Copy link

amarsman commented Sep 28, 2019

Enjoy!

Aschwin Marsman
@coveralls

This comment has been minimized.

Copy link

coveralls commented Sep 28, 2019

Coverage Status

Coverage decreased (-0.003%) to 6.938% when pulling 828ba47 on amarsman:master into df43cec on IJHack:master.

@codecov

This comment has been minimized.

Copy link

codecov bot commented Sep 28, 2019

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #478   +/-   ##
=====================================
  Coverage     7.3%   7.3%           
=====================================
  Files          44     44           
  Lines        2806   2806           
=====================================
  Hits          205    205           
  Misses       2601   2601
Impacted Files Coverage Δ
src/storemodel.h 0% <ø> (ø) ⬆️

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 df43cec...828ba47. Read the comment docs.

@annejan annejan merged commit a645baa into IJHack:master Sep 28, 2019
5 checks passed
5 checks passed
CodeFactor No issues found.
Details
codecov/patch Coverage not affected when comparing df43cec...828ba47
Details
codecov/project 7.3% remains the same compared to df43cec
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.