Closed eirikurj closed 1 year ago
Merging #76 (18f4528) into main (d696522) will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## main #76 +/- ##
=======================================
Coverage 73.43% 73.43%
=======================================
Files 6 6
Lines 753 753
=======================================
Hits 553 553
Misses 200 200
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
Ugh, right, the PATH
is probably missing the directory where the lib is. Will fix.
Purpose
Currently, the library import check suppresses useful information why the library could not be imported. This PR updates the import check to print out the full error message.
Expected time until merged
No rush.
Type of change
Testing
Compile the module with and without the fix, with a broken library (e.g., by commenting out
CGNS_LINKER_FLAGS
in the config).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