Skip to content

Conversation

devmotion
Copy link
Member

StatsFuns 0.9.10 (released yesterday) uses CR 1. It has 655 dependents (93 direct ones) (https://juliahub.com/ui/Packages/StatsFuns/530lR/0.9.10?t=2) and therefore I think it could be useful to add it to the integration tests.

@codecov-commenter
Copy link

codecov-commenter commented Aug 30, 2021

Codecov Report

Merging #451 (5dceedc) into master (8218c2c) will increase coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #451      +/-   ##
==========================================
+ Coverage   92.85%   92.88%   +0.02%     
==========================================
  Files          14       14              
  Lines         784      787       +3     
==========================================
+ Hits          728      731       +3     
  Misses         56       56              
Impacted Files Coverage Δ
src/projection.jl 98.09% <0.00%> (+0.02%) ⬆️

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 8218c2c...5dceedc. Read the comment docs.

Copy link
Member

@mcabbott mcabbott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good. Its tests take 2m 35s which is approximately instant.

@devmotion devmotion merged commit 3b46ac5 into master Aug 31, 2021
@devmotion devmotion deleted the dw/integrationtest_statsfuns branch August 31, 2021 00:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants