Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Use CRlibm 0.5 that does not export functions #238

Merged
merged 2 commits into from
Mar 6, 2017
Merged

Conversation

dpsanders
Copy link
Member

@dpsanders dpsanders commented Mar 6, 2017

CRlibm 0.5 (the latest release) no longer exports functions, so we define them explicitly in rounding.jl.

This is in preparation for #220, but should be included in v0.7.

@codecov-io
Copy link

codecov-io commented Mar 6, 2017

Codecov Report

Merging #238 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #238      +/-   ##
==========================================
+ Coverage   91.85%   91.86%   +<.01%     
==========================================
  Files          26       26              
  Lines        1044     1045       +1     
==========================================
+ Hits          959      960       +1     
  Misses         85       85
Impacted Files Coverage Δ
src/intervals/rounding.jl 76.47% <100%> (+1.47%)

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 a24a7f3...b28602a. Read the comment docs.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.008%) to 91.866% when pulling b28602a on use_crlibm_0_5 into a24a7f3 on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+0.008%) to 91.866% when pulling b28602a on use_crlibm_0_5 into a24a7f3 on master.

@dpsanders dpsanders merged commit dc436bc into master Mar 6, 2017
@dpsanders dpsanders deleted the use_crlibm_0_5 branch March 12, 2017 20:33
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants