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

Add more references between lowercase/uppercase operations. #39233

Merged
merged 1 commit into from
Jan 24, 2017

Conversation

frewsxcv
Copy link
Member

No description provided.

@frewsxcv
Copy link
Member Author

r? @GuillaumeGomez

@@ -77,6 +82,9 @@ pub trait AsciiExt {
/// assert_eq!('A', ascii.to_ascii_uppercase());
/// assert_eq!('❤', utf8.to_ascii_uppercase());
/// ```
///
/// [`make_ascii_uppercase`]: #tymethod.make_ascii_uppercase
/// [`str::to_uppercase`]: .../primitive.str.html#method.to_uppercase
Copy link
Member

Choose a reason for hiding this comment

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

"..."? ;)

@@ -96,6 +109,9 @@ pub trait AsciiExt {
/// assert_eq!('a', ascii.to_ascii_lowercase());
/// assert_eq!('❤', utf8.to_ascii_lowercase());
/// ```
///
/// [`make_ascii_lowercase`]: #tymethod.make_ascii_lowercase
/// [`str::to_lowercase`]: .../primitive.str.html#method.to_lowercase
Copy link
Member

Choose a reason for hiding this comment

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

"..."

@GuillaumeGomez
Copy link
Member

Nice idea! r=me once urls fixed.

@frewsxcv
Copy link
Member Author

@bors r=GuillaumeGomez rollup

@bors
Copy link
Contributor

bors commented Jan 22, 2017

📌 Commit d896a0c has been approved by GuillaumeGomez

@bors
Copy link
Contributor

bors commented Jan 22, 2017

⌛ Testing commit d896a0c with merge 965b698...

@bors
Copy link
Contributor

bors commented Jan 22, 2017

💔 Test failed - status-travis

@frewsxcv
Copy link
Member Author

@bors r+

🤖 🙅‍♂️

@bors
Copy link
Contributor

bors commented Jan 22, 2017

💡 This pull request was already approved, no need to approve it again.

  • This pull request previously failed. You should add more commits to fix the bug, or use retry to trigger a build again.
  • There's another pull request that is currently being tested, blocking this pull request: better comment wording #39238

@bors
Copy link
Contributor

bors commented Jan 22, 2017

📌 Commit d896a0c has been approved by frewsxcv

@frewsxcv
Copy link
Member Author

@bors r=GuillaumeGomez

Woops, didn't mean to r+ myself

@bors
Copy link
Contributor

bors commented Jan 22, 2017

💡 This pull request was already approved, no need to approve it again.

  • This pull request previously failed. You should add more commits to fix the bug, or use retry to trigger a build again.
  • There's another pull request that is currently being tested, blocking this pull request: better comment wording #39238

@bors
Copy link
Contributor

bors commented Jan 22, 2017

📌 Commit d896a0c has been approved by GuillaumeGomez

steveklabnik added a commit to steveklabnik/rust that referenced this pull request Jan 23, 2017
…umeGomez

Add more references between lowercase/uppercase operations.

None
bors added a commit that referenced this pull request Jan 23, 2017
Rollup of 7 pull requests

- Successful merges: #38794, #38956, #38993, #39191, #39200, #39233, #39258
- Failed merges:
@bors bors merged commit d896a0c into rust-lang:master Jan 24, 2017
@frewsxcv frewsxcv deleted the upper-lower-docs branch January 24, 2017 01:15
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.

3 participants