lanl-ansi / Juniper.jl

A JuMP-based Nonlinear Integer Program Solver
https://lanl-ansi.github.io/Juniper.jl/stable/
MIT License
180 stars 22 forks source link

use github actions as CI #208

Closed Wikunia closed 3 years ago

codecov[bot] commented 3 years ago

Codecov Report

Merging #208 (5d57029) into master (1511404) will decrease coverage by 0.18%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #208      +/-   ##
==========================================
- Coverage   92.33%   92.14%   -0.19%     
==========================================
  Files          22       22              
  Lines        2126     2012     -114     
==========================================
- Hits         1963     1854     -109     
+ Misses        163      158       -5     
Impacted Files Coverage Δ
src/bb_inits_and_defaults.jl 100.00% <100.00%> (ø)
src/MOI_wrapper/results.jl 80.76% <0.00%> (-3.11%) :arrow_down:
src/MOI_wrapper/variables.jl 73.23% <0.00%> (-1.77%) :arrow_down:
src/MOI_wrapper/constraints.jl 85.71% <0.00%> (-1.25%) :arrow_down:
src/util.jl 87.76% <0.00%> (-1.20%) :arrow_down:
src/MOI_wrapper/MOI_wrapper.jl 83.16% <0.00%> (-1.17%) :arrow_down:
src/printing.jl 78.94% <0.00%> (-0.60%) :arrow_down:
src/bb_strategies.jl 99.09% <0.00%> (-0.48%) :arrow_down:
src/bb_integral_or_branch.jl 96.77% <0.00%> (-0.37%) :arrow_down:
src/BnBTree.jl 93.43% <0.00%> (-0.26%) :arrow_down:
... and 10 more

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 d7a5187...5d57029. Read the comment docs.

ccoffrin commented 3 years ago

This looks reasonable to me. You might have a look at this commit that shows how we set this up in PowerModels.jl,

https://github.com/lanl-ansi/PowerModels.jl/commit/20e7cd0113ece0a3155d2783c8a5aff71514ed2a

TagBot can be updated to trigger as part of the CI action. We setup Documenter as a separate task, but I like your approach a little more.

Wikunia commented 3 years ago

I've added CompatHelper and changed Tagbot.