Clean up if clauses in Transform #1053

Closed
fnothaft opened this Issue Jun 17, 2016 · 0 comments

Comments

Projects
None yet
2 participants
@fnothaft
Member

fnothaft commented Jun 17, 2016

org.bdgenomics.adam.cli.Transform has a ton of if clauses that allow us to treat the Transform class essentially as a one-stop-shop for all your Read Transforming Needs™.

However, as has been rightly pointed out, this is a mess to read and follow. As such, we should clean up the tangled web of transforms in Transform.

@heuermh heuermh referenced this issue Jun 27, 2016

Closed

Release ADAM version 0.20.0 #1048

47 of 61 tasks complete

@heuermh heuermh modified the milestone: 0.20.0 Jul 18, 2016

fnothaft added a commit to fnothaft/adam that referenced this issue Jul 19, 2016

[ADAM-1053] Clean up if-clauses in Transform CLI.
Moves all if clauses into `maybeX` methods. Resolves #1053.

@heuermh heuermh closed this in #1081 Jul 19, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment