Closed timholy closed 2 years ago
Merging #102 (f45f403) into master (c2114a7) will decrease coverage by
1.28%
. The diff coverage isn/a
.:exclamation: Current head f45f403 differs from pull request most recent head 90e576e. Consider uploading reports for the commit 90e576e to get more accurate results
@@ Coverage Diff @@
## master #102 +/- ##
==========================================
- Coverage 96.86% 95.58% -1.29%
==========================================
Files 8 8
Lines 701 611 -90
==========================================
- Hits 679 584 -95
- Misses 22 27 +5
Impacted Files | Coverage Δ | |
---|---|---|
src/multinomials.jl | 90.00% <0.00%> (-2.31%) |
:arrow_down: |
src/partitions.jl | 97.96% <0.00%> (-2.04%) |
:arrow_down: |
src/permutations.jl | 98.01% <0.00%> (-1.99%) |
:arrow_down: |
src/combinations.jl | 83.48% <0.00%> (-1.79%) |
:arrow_down: |
src/youngdiagrams.jl | 97.50% <0.00%> (-0.23%) |
:arrow_down: |
src/numbers.jl | 100.00% <0.00%> (ø) |
|
src/factorials.jl | 100.00% <0.00%> (ø) |
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 c2114a7...90e576e. Read the comment docs.
@logankilpatrick looks like you have merge rights here. it'd be good to do this one, since it looks like CI hasn't been run in almost a year
@timholy does this still look okay to merge?
This pull request sets up the CI workflow on this repository.