Skip to content

doc: use the more convenient 'cargo run' command #16721

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

Merged
merged 2 commits into from
Aug 31, 2014
Merged

doc: use the more convenient 'cargo run' command #16721

merged 2 commits into from
Aug 31, 2014

Conversation

tshepang
Copy link
Member

This lovely command has already been introduced, so let's take advantage
of it.

This lovely command has already been introduced, so let's take advantage
of it.
@steveklabnik
Copy link
Member

Thanks! This part was written before 'cargo run' existed 😅

@tshepang
Copy link
Member Author

:-)

@alexcrichton
Copy link
Member

@steveklabnik, one thing I've noticed is that an r+ from a phone on a commit actually doesn't show up. If you look at this description page, it looks like it's there, but if you go and click through to the commit itself it doesn't show up.

I have no idea why :(

alexcrichton added a commit to alexcrichton/rust that referenced this pull request Aug 29, 2014
@alexcrichton
Copy link
Member

Sadly the documentation fails to build with this commit, could you make the same modifications (to the markdown files) as in alexcrichton@89163fc?

Addresses this comment:
#16721 (comment)
@tshepang
Copy link
Member Author

Not sure I understand, but I've added a commit.

alexcrichton added a commit to alexcrichton/rust that referenced this pull request Aug 31, 2014
@bors bors merged commit d89b2a5 into rust-lang:master Aug 31, 2014
@tshepang tshepang deleted the convenience branch May 31, 2015 00:04
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.

4 participants