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

Documentation cleanup and style improvements #492

Merged
merged 1 commit into from Nov 20, 2014

Conversation

Projects
None yet
4 participants
@massie
Member

massie commented Nov 19, 2014

No description provided.

@arahuja

This comment has been minimized.

Show comment
Hide comment
@arahuja

arahuja Nov 19, 2014

Contributor

The running_on_ec2 and running_on_cdh docs don't use adam-submit. Maybe not here, but we should update those right? There is more we can add to the CDH docs for running on YARN as well - I can add to that

Contributor

arahuja commented Nov 19, 2014

The running_on_ec2 and running_on_cdh docs don't use adam-submit. Maybe not here, but we should update those right? There is more we can add to the CDH docs for running on YARN as well - I can add to that

@massie

This comment has been minimized.

Show comment
Hide comment
@massie

massie Nov 19, 2014

Member

I've uploaded the PDF/html to the 0.14.0 release for reviewers to see.

https://github.com/bigdatagenomics/adam/releases/tag/adam-parent-0.14.0

Member

massie commented Nov 19, 2014

I've uploaded the PDF/html to the 0.14.0 release for reviewers to see.

https://github.com/bigdatagenomics/adam/releases/tag/adam-parent-0.14.0

@massie

This comment has been minimized.

Show comment
Hide comment
@massie

massie Nov 19, 2014

Member

@arahuja I was more focused on the presentation than content. Once this PR drops, can you open a PR with the corrected content?

Member

massie commented Nov 19, 2014

@arahuja I was more focused on the presentation than content. Once this PR drops, can you open a PR with the corrected content?

@AmplabJenkins

This comment has been minimized.

Show comment
Hide comment
@AmplabJenkins

AmplabJenkins Nov 19, 2014

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

AmplabJenkins commented Nov 19, 2014

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

@arahuja

This comment has been minimized.

Show comment
Hide comment
@arahuja

arahuja Nov 19, 2014

Contributor

Sure Matt - I opened that issue at #493 and will put in some updates on those

Contributor

arahuja commented Nov 19, 2014

Sure Matt - I opened that issue at #493 and will put in some updates on those

@massie

This comment has been minimized.

Show comment
Hide comment
@massie

massie Nov 19, 2014

Member

Thanks, Arun!

Member

massie commented Nov 19, 2014

Thanks, Arun!

@tdanford

This comment has been minimized.

Show comment
Hide comment
@tdanford

tdanford Nov 20, 2014

Contributor

@massie can you rebase this off of the latest master?

Contributor

tdanford commented Nov 20, 2014

@massie can you rebase this off of the latest master?

@massie

This comment has been minimized.

Show comment
Hide comment
@massie

massie Nov 20, 2014

Member

Rebased.

Member

massie commented Nov 20, 2014

Rebased.

@AmplabJenkins

This comment has been minimized.

Show comment
Hide comment
@AmplabJenkins

AmplabJenkins Nov 20, 2014

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

AmplabJenkins commented Nov 20, 2014

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

tdanford added a commit that referenced this pull request Nov 20, 2014

Merge pull request #492 from massie/doc-cleanup
Documentation cleanup and style improvements

@tdanford tdanford merged commit faa2863 into bigdatagenomics:master Nov 20, 2014

1 check passed

default Merged build finished.
Details
@tdanford

This comment has been minimized.

Show comment
Hide comment
@tdanford

tdanford Nov 20, 2014

Contributor

Thanks, @massie!

Contributor

tdanford commented Nov 20, 2014

Thanks, @massie!

@massie massie deleted the massie:doc-cleanup branch Nov 20, 2014

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