mdolab / idwarp

IDWarp is a mesh warping package for the MACH framework.

Other

17 stars 29 forks source link

Consolidate compile flags #59

Closed eirikurj closed 3 years ago

eirikurj commented 3 years ago

Purpose

Previously, flags were included twice when compiling. This PR consolidates and simplifies the flags used.

Type of change

What types of change is it? Select the appropriate type(s) that describe this PR

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (non-backwards-compatible fix or feature)
[ ] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no API changes)
[ ] Documentation update
[ ] Maintenance update
[ ] Other (please describe)

Testing

Tested locally.

Checklist

Put an x in the boxes that apply.

[ ] I have run flake8 and black to make sure the code adheres to PEP-8 and is consistently formatted
[x] I have run unit and regression tests which pass locally with my changes
[ ] I have added new tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation

codecov[bot] commented 3 years ago

Codecov Report

Merging #59 (d8eb2e1) into master (1139b7d) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #59   +/-   ##
=======================================
  Coverage   46.13%   46.13%           
=======================================
  Files           6        6           
  Lines         776      776           
=======================================
  Hits          358      358           
  Misses        418      418

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 1139b7d...d8eb2e1. Read the comment docs.