Fix/codecov experiment

[ivirshup]

Experimenting to get #71 working

[codecov]

Codecov Report

Merging #76 (c8b9087) into main (9aafb47) will increase coverage by 86.73%.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff             @@
##            main      #76       +/-   ##
==========================================
+ Coverage   0.00%   86.73%   +86.73%     
==========================================
  Files         16       16               
  Lines       1523     1523               
==========================================
+ Hits           0     1321     +1321     
+ Misses      1523      202     -1321     

Flag	Coverage Δ
unittests	?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
spatialdata/_types.py	62.50% <0.00%> (+62.50%)	⬆️
spatialdata/utils.py	68.57% <0.00%> (+68.57%)	⬆️
spatialdata/_io/format.py	76.08% <0.00%> (+76.08%)	⬆️
spatialdata/_core/models.py	78.40% <0.00%> (+78.40%)	⬆️
spatialdata/_core/coordinate_system.py	89.18% <0.00%> (+89.18%)	⬆️
spatialdata/_core/core_utils.py	91.56% <0.00%> (+91.56%)	⬆️
spatialdata/_logging.py	92.85% <0.00%> (+92.85%)	⬆️
spatialdata/_core/transformations.py	95.39% <0.00%> (+95.39%)	⬆️
spatialdata/_core/_spatialdata.py	95.97% <0.00%> (+95.97%)	⬆️
spatialdata/_io/write.py	96.58% <0.00%> (+96.58%)	⬆️
... and 4 more

[ivirshup]

@giovp, it takes a little bit for the coverage action to run, and it's not added to the checks until after it runs.

Maybe we should make it a required check?

[giovp]

do you think this made it work? or what was it otherwise?

[ivirshup]

Yes, it wasn't collecting the coverage otherwise. I can locally reproduce that installing without -e prevents collection of the coverage.

I vaguely remember this being an issue before. I think it came up for anndata – though it's not using -e now and I can't tell what's different 😅. There's also some reason we make the cookiecutter do an editable install too.

I think it had something to do with the install being a wheel, and being able to trace the code being tested to the wheel?

[giovp]

Maybe we should make it a required check?

we could make it a required check

[ivirshup]

I'll merge this then open an issue for getting coverage to run without doing an editable install