Consolidate compile flags

[eirikurj]

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)
• 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
• 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]

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.

[ewu63]

Looks good, I think we will want to do this for some other repos too right?