Skip to content

Conversation

aidanhs
Copy link
Contributor

@aidanhs aidanhs commented Jul 21, 2015

Continuation of #25685.

@rust-highfive
Copy link
Contributor

r? @alexcrichton

(rust_highfive has picked a reviewer for you, use r? to override)

Copy link
Contributor

Choose a reason for hiding this comment

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

rustc should be in graves

@steveklabnik
Copy link
Contributor

I have a bunch of nitpicks, but thanks so much for writing this up!

@aidanhs aidanhs force-pushed the aphs-advanced-linking-doc branch from 4068d72 to 844d81a Compare July 21, 2015 18:21
@aidanhs
Copy link
Contributor Author

aidanhs commented Jul 21, 2015

Think I got them all.

Copy link
Contributor

Choose a reason for hiding this comment

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

*dependent

@aidanhs aidanhs force-pushed the aphs-advanced-linking-doc branch from 844d81a to 231438a Compare July 21, 2015 20:12
@alexcrichton
Copy link
Member

@bors: r+ 231438a rollup

Nice!

steveklabnik added a commit to steveklabnik/rust that referenced this pull request Jul 21, 2015
Copy link
Contributor

Choose a reason for hiding this comment

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

This code block needs to be annotated so that it isn't compiled as Rust code. CC #27206.

@steveklabnik
Copy link
Contributor

@bors: r-

@steveklabnik
Copy link
Contributor

minusing this one until we fix the issues, we'll get it in soon!

@aidanhs aidanhs force-pushed the aphs-advanced-linking-doc branch from 231438a to b6a0d9e Compare July 22, 2015 17:15
@aidanhs
Copy link
Contributor Author

aidanhs commented Jul 22, 2015

Added 'text' to code blocks.

@steveklabnik
Copy link
Contributor

@bors: r+ rollup

@bors
Copy link
Collaborator

bors commented Jul 22, 2015

📌 Commit b6a0d9e has been approved by steveklabnik

Manishearth added a commit to Manishearth/rust that referenced this pull request Jul 23, 2015
steveklabnik added a commit to steveklabnik/rust that referenced this pull request Jul 24, 2015
bors added a commit that referenced this pull request Jul 24, 2015
@bors bors merged commit b6a0d9e into rust-lang:master Jul 24, 2015
@aidanhs
Copy link
Contributor Author

aidanhs commented Jul 25, 2015

Hooray! Thanks all for the help with this.

@aidanhs aidanhs deleted the aphs-advanced-linking-doc branch July 25, 2015 11:51
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.

7 participants