-
Notifications
You must be signed in to change notification settings - Fork 11
Conversation
It would be nice if somebody reviewed #167 so we can merge that, rebase this and then merge this to fix all CI issues popping up: |
|
@jbgi Good point, I'll put that back. |
Added |
Why isn't Coveralls working? 🤔 Otherwise, I have no clue why it's not working. |
I've managed to run the test locally so if coveralls are fixed, LGTM |
We can leave the coveralls issue for another task, not urgent. |
@ksaric Seems that 30829cc#diff-8b496edab86012a437e80897eb7129c0 removed coverage checking. |
@365andreas Yep, I noticed that. Adding it here results in:
|
#170
Fix the failing CI and remove unnecessary Nix derivations.