Update to AlignmentRecordField and its usages as contig changed to co… #992

Merged
merged 1 commit into from Apr 7, 2016

Conversation

Projects
None yet
5 participants
@jpdna
Member

jpdna commented Apr 7, 2016

…ntigName in alignmentRecord

Address #991 raised by @erictu

updates AlignmentRecordField contig and mateContig to contigName and mateContigName as required by recent commit 7823abd

@AmplabJenkins

This comment has been minimized.

Show comment
Hide comment
@AmplabJenkins

AmplabJenkins Apr 7, 2016

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/1146/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/1146/
Test PASSed.

@heuermh

This comment has been minimized.

Show comment
Hide comment
@heuermh

heuermh Apr 7, 2016

Member

+1

Member

heuermh commented Apr 7, 2016

+1

@jpdna jpdna referenced this pull request in bigdatagenomics/mango Apr 7, 2016

Closed

Possible bdg-formats seq-dict related bug #144

@fnothaft

This comment has been minimized.

Show comment
Hide comment
@fnothaft

fnothaft Apr 7, 2016

Member

LGTM!

Member

fnothaft commented Apr 7, 2016

LGTM!

@fnothaft fnothaft merged commit 500c1cb into bigdatagenomics:master Apr 7, 2016

1 check passed

default Merged build finished.
Details
@fnothaft

This comment has been minimized.

Show comment
Hide comment
@fnothaft

fnothaft Apr 7, 2016

Member

Merged! Thanks @jpdna!

Member

fnothaft commented Apr 7, 2016

Merged! Thanks @jpdna!

@erictu

This comment has been minimized.

Show comment
Hide comment
@erictu

erictu Apr 7, 2016

Member

Just tested it downstream, works fine!

Member

erictu commented Apr 7, 2016

Just tested it downstream, works fine!

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