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

DrWavAudioImporter: update dr_wav to 0.8.5. #61

Merged
merged 2 commits into from Jun 1, 2019

Conversation

3 participants
@williamjcm
Copy link
Contributor

commented Jun 1, 2019

Link to the corresponding commit/revision: mackron/dr_libs@5e8fd96

The tests were also updated.

Review/feedback welcome!

@mosra

mosra approved these changes Jun 1, 2019

Copy link
Owner

left a comment

All looks good from my side 👍 💯 Let's see what the CIs say, this is ready to get merged from my side.

@mosra mosra added this to the 2019.0b milestone Jun 1, 2019

@mosra mosra added this to TODO in Asset management via automation Jun 1, 2019

@codecov-io

This comment has been minimized.

Copy link

commented Jun 1, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #61   +/-   ##
=======================================
  Coverage   90.89%   90.89%           
=======================================
  Files          49       49           
  Lines        4227     4227           
=======================================
  Hits         3842     3842           
  Misses        385      385

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 382a72b...6c13d00. Read the comment docs.

@mosra mosra merged commit 6c13d00 into mosra:master Jun 1, 2019

4 checks passed

codecov/patch Coverage not affected when comparing 382a72b...6c13d00
Details
codecov/project 90.89% remains the same compared to 382a72b
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

Asset management automation moved this from TODO to Done Jun 1, 2019

@mosra

This comment has been minimized.

Copy link
Owner

commented Jun 1, 2019

Merged, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.