Skip to content
This repository was archived by the owner on Oct 9, 2018. It is now read-only.

Conversation

@edunham
Copy link

@edunham edunham commented May 27, 2015

The rustbook link redirects, so I put the new one in.

Also put the build command in the section that promises it.

The rustbook link redirects, so I put the new one in. Also put the build command in the section that promises it.
@rust-highfive
Copy link

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @nikomatsakis (or someone else) soon.

If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. The way Github handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.

Please see CONTRIBUTING.md for more information.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@steveklabnik do you know if your rustbook repo still builds? If not this may want to actually link to the in-tree copy as it's what's actually building the documentation in-tree at least. (not a great situation to be in!)

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The CI badge on the repo says its build is currently failing...

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My repo is just a copy of the in-tree code. I thought I had rust-ci letting me know if it was broke, but maybe I don't.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah ok, perhaps this could link to the in-tree copy then? Other than that looks good to me!

@edunham edunham closed this May 29, 2015
@edunham edunham deleted the patch-1 branch May 29, 2015 19:47
@steveklabnik
Copy link
Contributor

Oh hey sorry! I didn't realize this was open against this repo :(

@edunham
Copy link
Author

edunham commented May 29, 2015

Don't worry, this PR's goal got accomplished even better than I expected -- it drew enough attention to the repo to point out that it was redundant :)

@steveklabnik
Copy link
Contributor

:)

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

Labels

None yet

5 participants