-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CI improvements #100
CI improvements #100
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #100 +/- ##
==========================================
- Coverage 43.66% 43.63% -0.03%
==========================================
Files 17 17
Lines 1152 1155 +3
==========================================
+ Hits 503 504 +1
- Misses 649 651 +2 ☔ View full report in Codecov by Sentry. |
@ggmarshall can you spot the problem? If you fix it then it's good to merge. Unfortunately But is touching output files smart? Do we code in the Snakefiles that needs to interact with non-dummy outputs? |
I have to implement something to make the test output more readable... |
@ggmarshall what about now? |
this is tougher to fix as those aren't stored on git, I want to remove them soon |
let's just touch them? |
Sure let’s do that for the moment |
I want to remove them eventually though |
I'm gonna merge this since tests pass, we can find a way to speed up the thing with a future PR |
No description provided.