Skip to content
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

bamcheck fix for projects with -d or -t in their path #16

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

mp15
Copy link
Contributor

@mp15 mp15 commented Nov 28, 2013

Change bamcheck options so that targeted is passed via command line and not regexed out as this causes problems

@sb10
Copy link
Member

sb10 commented Nov 28, 2013

Thanks for this Martin.

Do you know of a test script that utilises this code path, or can be easily made to do so?

Also, am I right in thinking that this change is backwards compatible in the sense that if an existing setup has '-d' in bamcheck_options, it will continue to work just fine?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants