KnutAM / FESolvers.jl

Solvers for Ferrite.jl problems
https://knutam.github.io/FESolvers.jl/dev
MIT License
3 stars 0 forks source link

Follow julia-base convention #14

Closed KnutAM closed 1 year ago

KnutAM commented 1 year ago

By mistake the original definition of solve_problem! was solve_problem!(solver, problem), but following Julia's Style Guide it should be reversed. Better do it now than later... I.e. change to solve_problem!(problem, solver) Will bump the version soon after merging this...

codecov-commenter commented 1 year ago

Codecov Report

Merging #14 (4ebfdff) into main (dc473f6) will not change coverage. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #14   +/-   ##
=======================================
  Coverage   94.47%   94.47%           
=======================================
  Files           8        8           
  Lines         163      163           
=======================================
  Hits          154      154           
  Misses          9        9           
Impacted Files Coverage Δ
src/FESolvers.jl 100.00% <ø> (ø)
src/QuasiStaticSolver.jl 95.83% <100.00%> (ø)
src/nlsolvers.jl 97.91% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more