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

OvrIntegration: adapt to Magnum::GL API changes #32

merged 1 commit into from Aug 31, 2018


3 participants

Squareys commented Aug 30, 2018

Hi @mosra !

As per gitter.

Cheers, Jonathan :)

OvrIntegration: adapt to Magnum::GL API changes
Signed-off-by: Squareys <>

This comment has been minimized.

codecov-io commented Aug 31, 2018

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #32   +/-   ##
  Coverage   58.51%   58.51%           
  Files          16       16           
  Lines         552      552           
  Hits          323      323           
  Misses        229      229

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 968fa40...15140dc. Read the comment docs.

@mosra mosra added this to the 2018.0c milestone Aug 31, 2018

@mosra mosra added this to TODO in VR via automation Aug 31, 2018

@mosra mosra merged commit 15140dc into mosra:master Aug 31, 2018

4 checks passed

codecov/patch Coverage not affected when comparing 968fa40...15140dc
codecov/project 58.51% remains the same compared to 968fa40
continuous-integration/appveyor/pr AppVeyor build succeeded
continuous-integration/travis-ci/pr The Travis CI build passed

VR automation moved this from TODO to Done Aug 31, 2018


This comment has been minimized.


mosra commented Aug 31, 2018

Awesome. Thank you a lot.


This comment has been minimized.


Squareys commented Aug 31, 2018

Thanks for merging! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment