Closed hajdik closed 1 year ago
Merging #164 (a941087) into main (2905061) will decrease coverage by
0.00%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## main #164 +/- ##
==========================================
- Coverage 63.93% 63.92% -0.01%
==========================================
Files 47 47
Lines 11819 11818 -1
==========================================
- Hits 7556 7555 -1
Misses 4263 4263
Impacted Files | Coverage Δ | |
---|---|---|
pygeo/constraints/baseConstraint.py | 57.22% <ø> (-0.24%) |
:arrow_down: |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
It looks like we need to pin one of flake8
sub packages?
It's a valid complaint, I think we should probably fix it.
How are we taking care of the flake8 fixes?
I removed addVariablesPyOpt
from baseConstraint
because only ColinearityConstraint
implements this method (which makes sense because most constraints don't have their own DVs).
Purpose
The readme didn't have all the parameterization options or the right year
Expected time until merged
fast
Type of change
Checklist
flake8
andblack
to make sure the Python code adheres to PEP-8 and is consistently formattedfprettify
or C/C++ code withclang-format
as applicable