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

[ADAM-1453] Add hooks to publish ADAM docs from CI flow. #1461

Merged
merged 1 commit into from Mar 31, 2017

Conversation

@fnothaft
Copy link
Member

fnothaft commented Mar 30, 2017

Resolves #1453.

@fnothaft fnothaft requested a review from heuermh Mar 30, 2017
@fnothaft fnothaft added this to the 0.22.0 milestone Mar 30, 2017
@fnothaft
Copy link
Member Author

fnothaft commented Mar 30, 2017

@heuermh ping me before you merge this, because it will require a small change to @AmplabJenkins. Essentially, we just need to sequence this/the Jenkins change and #1412.

@coveralls
Copy link

coveralls commented Mar 30, 2017

Coverage Status

Coverage remained the same at 80.858% when pulling 362b4aa on fnothaft:issues/1453-publish-docs into 65d8c9e on bigdatagenomics:master.

@AmplabJenkins
Copy link

AmplabJenkins commented Mar 30, 2017

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

@heuermh
Copy link
Member

heuermh commented Mar 31, 2017

Resolves #774

@fnothaft
Copy link
Member Author

fnothaft commented Mar 31, 2017

@AmplabJenkins is updated; this is good to go from my side.

@heuermh heuermh merged commit 462db97 into bigdatagenomics:master Mar 31, 2017
3 checks passed
3 checks passed
codacy/pr Good work! A positive pull request.
Details
coverage/coveralls Coverage remained the same at 80.858%
Details
default Merged build finished.
Details
@heuermh
Copy link
Member

heuermh commented Mar 31, 2017

Thank you, @fnothaft

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

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.