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

Build the book on CI #114

Merged
merged 3 commits into from
Sep 3, 2019
Merged

Build the book on CI #114

merged 3 commits into from
Sep 3, 2019

Conversation

lqd
Copy link
Member

@lqd lqd commented Aug 30, 2019

Try to build the mdbook on CI and autodeploy it from there

@lqd
Copy link
Member Author

lqd commented Aug 30, 2019

According to the doc this should be how it's done, what do you think @pietroalbini ? :)

@lqd lqd changed the title [WIP] Build the book on CI Build the book on CI Aug 30, 2019
@pietroalbini pietroalbini self-assigned this Aug 30, 2019
This was referenced Aug 31, 2019
@pietroalbini
Copy link
Member

We have a different deployment code for the rust-lang org. You need to replace the deploy: block with:

env:
  RUSTINFRA_DEPLOY_DIR: book/book
import:
  - rust-lang/simpleinfra/travis-configs/static-websites.yml

@lqd
Copy link
Member Author

lqd commented Sep 3, 2019

Ah thank you, I thought it was somewhat specific to hugo (as in the compiler-team repo), or at the very least unrelated to mdbook deployments since the set-up seemed different at first glance in the rustc-guide repo.

I'll change this ASAP.

@pietroalbini
Copy link
Member

That piece of infra is relatively new, so it's not surprising most of our repositories don't use it.

@lqd
Copy link
Member Author

lqd commented Sep 3, 2019

Thanks a lot for the help @pietroalbini !

I guess it's time to ... try ? :)

@lqd lqd merged commit f52fd71 into rust-lang:master Sep 3, 2019
@lqd lqd deleted the book branch September 3, 2019 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants