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 links in RELEASES.md for 1.10.0 through 1.12.0 #41613

Merged
merged 1 commit into from
Apr 30, 2017

Conversation

cuviper
Copy link
Member

@cuviper cuviper commented Apr 28, 2017

Many links in this series have the [link text] and (url) on separate
lines, which doesn't get correctly interpreted in markdown. Or maybe it
once did, but it doesn't now. This patch joins the lines together.

Here is the content rendered before and after.

Many links in this series have the `[link text]` and `(url)` on separate
lines, which doesn't get correctly interpreted in markdown.  Or maybe it
once did, but it doesn't now.  This patch joins the lines together.
@rust-highfive
Copy link
Collaborator

r? @aturon

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

@cuviper
Copy link
Member Author

cuviper commented Apr 28, 2017

If you don't like the length of the joined lines, another alternative that still avoids spurious whitespace is to use HTML comments:

[link text<!--
-->](url)

A better change is to use out-of-line urls like all the other versions do, but that's a very manual process.

@aturon
Copy link
Member

aturon commented Apr 29, 2017

Awesome, thanks!

@bors: r+ rollup

@bors
Copy link
Contributor

bors commented Apr 29, 2017

📌 Commit e8c4b7a has been approved by aturon

@steveklabnik
Copy link
Member

steveklabnik commented Apr 29, 2017 via email

frewsxcv added a commit to frewsxcv/rust that referenced this pull request Apr 29, 2017
Fix links in RELEASES.md for 1.10.0 through 1.12.0

Many links in this series have the `[link text]` and `(url)` on separate
lines, which doesn't get correctly interpreted in markdown.  Or maybe it
once did, but it doesn't now.  This patch joins the lines together.

Here is the content rendered [before](https://github.com/rust-lang/rust/blob/2971d491b955fd8bb504c8ca675b8116b8b1550e/RELEASES.md#version-1120-2016-09-29) and [after](https://github.com/cuviper/rust/blob/e8c4b7af216c9e035b40f78cbd77a35614fdc240/RELEASES.md#version-1120-2016-09-29).
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Apr 30, 2017
Fix links in RELEASES.md for 1.10.0 through 1.12.0

Many links in this series have the `[link text]` and `(url)` on separate
lines, which doesn't get correctly interpreted in markdown.  Or maybe it
once did, but it doesn't now.  This patch joins the lines together.

Here is the content rendered [before](https://github.com/rust-lang/rust/blob/2971d491b955fd8bb504c8ca675b8116b8b1550e/RELEASES.md#version-1120-2016-09-29) and [after](https://github.com/cuviper/rust/blob/e8c4b7af216c9e035b40f78cbd77a35614fdc240/RELEASES.md#version-1120-2016-09-29).
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Apr 30, 2017
Fix links in RELEASES.md for 1.10.0 through 1.12.0

Many links in this series have the `[link text]` and `(url)` on separate
lines, which doesn't get correctly interpreted in markdown.  Or maybe it
once did, but it doesn't now.  This patch joins the lines together.

Here is the content rendered [before](https://github.com/rust-lang/rust/blob/2971d491b955fd8bb504c8ca675b8116b8b1550e/RELEASES.md#version-1120-2016-09-29) and [after](https://github.com/cuviper/rust/blob/e8c4b7af216c9e035b40f78cbd77a35614fdc240/RELEASES.md#version-1120-2016-09-29).
bors added a commit that referenced this pull request Apr 30, 2017
Rollup of 6 pull requests

- Successful merges: #41449, #41509, #41608, #41613, #41636, #41637
- Failed merges:
@bors bors merged commit e8c4b7a into rust-lang:master Apr 30, 2017
@cuviper cuviper deleted the fix-release-links branch September 26, 2017 06:39
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.

5 participants