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

only deploy docs when tagged (fix #18) #30

Merged
merged 1 commit into from Aug 11, 2016

Conversation

Projects
None yet
2 participants
@benaryorg
Contributor

benaryorg commented Aug 9, 2016

Docs are still built (not uploaded) when untagged, but this might reveal bugs so I didn't change it.

I also don't have any means of testing this right now but I more or less copied from the travis docs so it is supposed to work.

only deploy docs when tagged (fix #18)
Signed-off-by: benaryorg <binary@benary.org>
@benaryorg

This comment has been minimized.

Show comment
Hide comment
@benaryorg

benaryorg Aug 9, 2016

Contributor

Okay, travis looks fine. We'll have to wait for the next release I guess.

Oh, and do not tag old things. This will behave the same as it did without this patch, it will start the travis build and upload the old stuff.

Contributor

benaryorg commented Aug 9, 2016

Okay, travis looks fine. We'll have to wait for the next release I guess.

Oh, and do not tag old things. This will behave the same as it did without this patch, it will start the travis build and upload the old stuff.

@brson brson merged commit f4837ee into rust-lang-nursery:master Aug 11, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@brson

This comment has been minimized.

Show comment
Hide comment
@brson

brson Aug 11, 2016

Contributor

Thanks!

Contributor

brson commented Aug 11, 2016

Thanks!

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