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
Remove unused val from Reads2Ref #458
Conversation
heh, did you find this because this line is failing tests on my predicates PR #457 ? |
Yes. Thanks for catching the bug. |
funny. I'm worried that it was a real bug in my predicate logic though, so still trying to debug that; doesn't need to hold up this PR though of course |
Test PASSed. |
Actually, there's more unused code. Cleaning it up now. |
Test PASSed. |
This is ready for someone to review. Should be an easy review -- just deleting code that isn't used. |
lgtm, fwiw! |
Jenkins, retest this please. |
Test PASSed. |
@massie will you rebase this? And then one of us (including me) would be happy to merge. |
Test PASSed. |
Remove unused val from Reads2Ref
Merged! Thanks @massie! |
No description provided.