Skip to content

Conversation

QuietMisdreavus
Copy link
Contributor

There was a comment today in IRC where someone thought char_indices() and chars().enumerate() were equivalent, so i wanted to put an example in the docs where that wasn't true.

r? @rust-lang/docs

@kennytm kennytm added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. A-docs Area: Documentation for any part of the project, including the compiler, standard library, and tools labels Dec 28, 2017
@frewsxcv
Copy link
Contributor

looks great! good idea

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Dec 28, 2017

📌 Commit ac15a2e has been approved by frewsxcv

@bors
Copy link
Collaborator

bors commented Dec 28, 2017

⌛ Testing commit ac15a2e with merge 77e189c...

bors added a commit that referenced this pull request Dec 28, 2017
update char_indices example to highlight big chars

There was a comment today in IRC where someone thought `char_indices()` and `chars().enumerate()` were equivalent, so i wanted to put an example in the docs where that wasn't true.

r? @rust-lang/docs
@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 Dec 28, 2017
@bors
Copy link
Collaborator

bors commented Dec 28, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: frewsxcv
Pushing 77e189c to master...

@bors bors merged commit ac15a2e into rust-lang:master Dec 28, 2017
@QuietMisdreavus QuietMisdreavus deleted the i-like-big-chars branch February 26, 2018 23:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-docs Area: Documentation for any part of the project, including the compiler, standard library, and tools 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.

4 participants