-
Notifications
You must be signed in to change notification settings - Fork 33
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
Y24-181 - Clean up HiSeq submission templates #4232
Y24-181 - Clean up HiSeq submission templates #4232
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #4232 +/- ##
===========================================
+ Coverage 86.74% 87.41% +0.67%
===========================================
Files 1369 1369
Lines 29746 29600 -146
===========================================
+ Hits 25802 25876 +74
+ Misses 3944 3724 -220 ☔ View full report in Codecov by Sentry. |
Code cov doesn't seem too happy with this but the behaviour is tested. I think because we are testing the output of the rake task rather than the internals it is not correctly identifying which parts have been tested. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
649f537
to
cb446cf
Compare
…on templates in seed data
…r and updates tests
- Updates disable hiseq submission rake task to ignore already superceded templates
…y with existing templates
aaba17f
to
2446b5d
Compare
Closes #4189
Changes proposed in this pull request