Add link to collections docs to extend trait

[skaunov]

I believe it would be useful here.

[rustbot]

r? @joshtriplett

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

[rustbot]

Hey! It looks like you've submitted a new PR for the library teams!

If this PR contains changes to any rust-lang/rust public library APIs then please comment with @rustbot label +T-libs-api -T-libs to tag it appropriately. If this PR contains changes to any unstable APIs please edit the PR description to add a link to the relevant API Change Proposal or create one if you haven't already. If you're unsure where your change falls no worries, just leave it as is and the reviewer will take a look and make a decision to forward on if necessary.

Examples of T-libs-api changes:

• Stabilizing library features
• Introducing insta-stable changes such as new implementations of existing stable traits on existing stable types
• Introducing new or changing existing unstable library APIs (excluding permanently unstable features / features without a tracking issue)
• Changing public documentation in ways that create new stability guarantees
• Changing observable runtime behavior of library APIs

[skaunov]

The job mingw-check-tidy failed! Check out the build log: (web) (plain)
Click to see the possible cause of the failure (guessed by this bot)

I guess it requires to run cargo fmt. =( Will do when reclaim enough free space at my disk to checkout locally.

[ChrisDenton]

While you're there, do you think linking collect to Iterator::collect would also be useful?

[skaunov]

Sure! Frankly, I would also link std::iter high-level page in the beginning of the https://doc.rust-lang.org/stable/std/collections/index.html#iterators section. Do you think it's appropriate?

While you're there

[ChrisDenton]

That definitely makes sense to me. Feel free to try it and I'll see how it looks.

[ChrisDenton]

Ok, this is looking great. The only thing is there are now quite a few commits. Would you be able to squish them into one?

[skaunov]

Sure. Just let me clone it again locally a bit later.

[skaunov]

Thank you for bearing with me on this one! Can I ask a side question while CI checks squashing...

Why do I get the following output when command cargo fmt in the module dir? What's the trick to make it work like in simpler crates? =(

`cargo metadata` exited with an error: error: failed to load manifest for workspace member `/home/serge/Desktop/rust/src/tools/cargo`

Caused by:
  failed to read `/home/serge/Desktop/rust/src/tools/cargo/Cargo.toml`

Caused by:
  No such file or directory (os error 2)
cargo-fmt 
...

[ChrisDenton]

Ah, Rust's build system is fairly complex and complicated further by the fact it's building the compiler and tools that are used to build itself. If that makes sense.

So you have to use the x command instead. E.g. ./x fmt. See the dev guide.

[ChrisDenton]

If you're interested in some of the details of the compiler's build system then Joshua Nelson's conference talk may be of interest: https://www.youtube.com/watch?v=oUIjG-y4zaA

[ChrisDenton]

Ok, let's try to get this merged!

@bors r+ rollup

[bors]

📌 Commit 18ca509 has been approved by ChrisDenton

It is now in the queue for this repository.

[skaunov]

Totally! I was suspecting something like this.

Ah, Rust's build system is fairly complex and complicated further by the fact it's building the compiler and tools that are used to build itself. If that makes sense.

I was cautiously looking at this py file, but was afraid to sink in different nuances. X))

So you have to use the x command instead. E.g. ./x fmt. See the dev guide.

Thanks a lot: ton of new stuff and information!