-
Notifications
You must be signed in to change notification settings - Fork 308
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
[ADAM-625] Enable globbing for BAM. #626
Conversation
Test FAILed. Build result: FAILUREGitHub pull request #626 of commit d48f1f1 automatically merged.Notifying endpoint 'HTTP:https://webhooks.gitter.im/e/ac8bb6e9f53357bc8aa8'[EnvInject] - Loading node environment variables.Building remotely on amp-jenkins-worker-05 (centos) in workspace /home/jenkins/workspace/ADAM-prb > git rev-parse --is-inside-work-tree # timeout=10Fetching changes from the remote Git repository > git config remote.origin.url https://github.com/bigdatagenomics/adam.git # timeout=10Fetching upstream changes from https://github.com/bigdatagenomics/adam.git > git --version # timeout=10 > git fetch --tags --progress https://github.com/bigdatagenomics/adam.git +refs/pull/:refs/remotes/origin/pr/ > git rev-parse origin/pr/626/merge^{commit} # timeout=10 > git branch -a --contains fd8f6a82ffc71c3b3d6b69977cb220f55a5e9b29 # timeout=10 > git rev-parse remotes/origin/pr/626/merge^{commit} # timeout=10Checking out Revision fd8f6a82ffc71c3b3d6b69977cb220f55a5e9b29 (origin/pr/626/merge) > git config core.sparsecheckout # timeout=10 > git checkout -f fd8f6a82ffc71c3b3d6b69977cb220f55a5e9b29First time build. Skipping changelog.Triggering ADAM-prb ? 2.2.0,centosTriggering ADAM-prb ? 2.3.0,centosTriggering ADAM-prb ? 1.0.4,centosADAM-prb ? 2.2.0,centos completed with result FAILUREADAM-prb ? 2.3.0,centos completed with result FAILUREADAM-prb ? 1.0.4,centos completed with result FAILURENotifying endpoint 'HTTP:https://webhooks.gitter.im/e/ac8bb6e9f53357bc8aa8'Test FAILed. |
@Args4jOption(required = false, name = "-force_load_fastq", usage = "Forces Transform to load from unpaired FASTQ.") | ||
var forceLoadFastq: Boolean = false | ||
@Args4jOption(required = false, name = "-force_load_fastq", usage = "Forces Transform to load from interleaved FASTQ.") | ||
var forceLoadIFastq: Boolean = false |
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.
did you mean fasta? or just one extra cmd+p?
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.
There was a missing i --> interleaved
; I had fixed it locally but forgot to commit before pushing.
lgtm after the build is fixed |
Test PASSed. |
Thanks, Frank! |
[ADAM-625] Enable globbing for BAM.
Resolves #625. Elaborates the glob before loading the header.