search

CGATOxford / UMI-tools

Tools for handling Unique Molecular Identifiers in NGS data sets
MIT License
491 stars 190 forks source link

Ts add test action #546

Closed TomSmithCGAT closed 2 years ago

TomSmithCGAT commented 2 years ago

Adds github actions to perform testing.

Still relying on the same bespoke test script we repurposed from cgat which utilises nose. Note that nose is no longer supported and cannot be used to test python 3.10 (https://github.com/nose-devs/nose/issues/1099). I tried a few of the suggestions in that issue to resolve the issue but nothing worked straight out of the box.

For now, I've dropped testing for python 3.10 in favour of getting the CI testing back up and running. In the long run, it seems like we should be switching to pytest.

TomSmithCGAT commented 2 years ago

No, was just doing my usual copy/paste/edit/debug and hadn't considered the minimum python version we'd like to test. How low to go?..

IanSudbery commented 2 years ago

I don't think many people are using 3.5 anymore, but 3.6 is still quite common I think.