search

lina-usc / pylossless

🧠 EEG Processing pipeline that annotates continuous data
https://pylossless.readthedocs.io/en/latest/
MIT License
20 stars 8 forks source link

Feature Request: We need basic tests #39

Closed scott-huberty closed 10 months ago

scott-huberty commented 1 year ago

We need to set up a basic CI protocol that manages flakes/docstyle/and some basic tests. We are at a point where we need to make sure our new PR's aren't creating code regression.

scott-huberty commented 1 year ago

50 was merged so this is at least started - but I'll close this issue once we have 1 test for each pipeline method.

scott-huberty commented 1 year ago

Maintenance note:

We are nearing 70% coverage - I need to update the github actions workflow to run on main so that our CodeCov badge is updated.

Once we are above 70%, I am comfortable with closing this meta issue, and more targeted tickets can be opened for specific modules/funcs that need a test.