search

lanl-ansi / Juniper.jl

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

Apply JuliaFormatter #233

Closed odow closed 2 years ago

odow commented 2 years ago

Closes #223

The PR #232 had a significant amount of whitespace changes that made it hard to read. Applying a consistent formatting should improve maintainability.

codecov[bot] commented 2 years ago

Codecov Report

Merging #233 (3da1916) into master (9b1c385) will decrease coverage by 0.50%. The diff coverage is 90.73%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #233      +/-   ##
==========================================
- Coverage   92.31%   91.81%   -0.51%     
==========================================
  Files          20       20              
  Lines        1991     2015      +24     
==========================================
+ Hits         1838     1850      +12     
- Misses        153      165      +12
Impacted Files Coverage Δ
src/MOI_wrapper/nlp.jl 0.00% <ø> (ø)
src/util.jl 71.17% <58.97%> (-2.61%) :arrow_down:
src/printing.jl 78.04% <61.11%> (ø)
src/MOI_wrapper/MOI_wrapper.jl 87.01% <77.58%> (-1.82%) :arrow_down:
src/fpump.jl 89.66% <91.25%> (+0.04%) :arrow_up:
src/solver.jl 79.76% <91.66%> (ø)
src/table_log.jl 95.31% <93.18%> (ø)
src/BnBTree.jl 94.64% <95.55%> (+0.03%) :arrow_up:
src/Juniper.jl 100.00% <100.00%> (ø)
src/MOI_wrapper/results.jl 88.37% <100.00%> (ø)
... and 13 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 9b1c385...3da1916. Read the comment docs.

odow commented 2 years ago

Merging because I think this is uncontroversial, and I'd like to make other changes to the tests without the formatting diffs getting in the way.

ccoffrin commented 2 years ago

Fine by me!