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

Fix most rendering warnings from switching to CommonMark #45419

Merged
merged 1 commit into from
Oct 21, 2017

Conversation

steveklabnik
Copy link
Member

There's one big one lift, I'm filing a bug for it soon.

r? @rust-lang/docs

@steveklabnik
Copy link
Member Author

cc @alexcrichton who noticed this

@steveklabnik
Copy link
Member Author

Other bug is #45420

@QuietMisdreavus
Copy link
Member

r=me pending travis. apparently the footnote issue was fixed in a later version of pulldown, so it's just a matter of updating rustdoc to point at a version that contains that fix. I opened #45421 to do that.

@kennytm kennytm added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 20, 2017
@QuietMisdreavus
Copy link
Member

The travis logs confirm that the only warnings left in std are connected to the footnote in libcore's crate-root docs. After double-checking against the latest commit in #45421, i'm reasonably sure that that should grab those warnings.

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Oct 20, 2017

📌 Commit f8f9005 has been approved by QuietMisdreavus

@@ -22,7 +22,7 @@ use mem;
/// This is currently the default hashing function used by standard library
/// (eg. `collections::HashMap` uses it by default).
///
/// See: https://131002.net/siphash/
/// See: <https://131002.net/siphash>
Copy link
Contributor

Choose a reason for hiding this comment

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

Nitpick: inconsistent trailing slash.

Copy link
Member Author

Choose a reason for hiding this comment

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

both work

Copy link
Member

Choose a reason for hiding this comment

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

The version without the trailing slash results in a 301 redirect.

@kennytm kennytm added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 21, 2017
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Oct 21, 2017
…ngs, r=QuietMisdreavus

Fix most rendering warnings from switching to CommonMark

There's one big one lift, I'm filing a bug for it soon.

r? @rust-lang/docs
bors added a commit that referenced this pull request Oct 21, 2017
Rollup of 6 pull requests

- Successful merges: #45227, #45356, #45407, #45411, #45418, #45419
- Failed merges: #45421
@bors bors merged commit f8f9005 into rust-lang:master Oct 21, 2017
@steveklabnik steveklabnik deleted the fix-commonmark-renderings branch October 25, 2017 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants