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

Update CONTRIBUTING.md #82887

Merged
merged 2 commits into from
Mar 9, 2021
Merged

Conversation

henryboisdequin
Copy link
Contributor

@henryboisdequin henryboisdequin commented Mar 8, 2021

Fixes #77215

As mentioned in #77215, the current CONTRIBUTING.md links to the rustc-dev-guide.
Even though the rustc-dev-guide has lots of useful information for contributors,
one is already confused by reading the first line of the current CONTRIBUTING.md.

To get started, read the [Getting Started] guide in the [rustc-dev-guide].

This line tells the contributor to go and read the rustc-dev-guide. What is
the rustc-dev-guide? What does rustc even mean? These are some of the
questions that went into my head when reading this line as a first-time
contributor. By explaining what the rustc-dev-guide is and some platforms
to get help, a new contributor understands what the first step is and the process
is much clearer. The About the [rustc-dev-guide] section explains what
the rustc-dev-guide is, what rustc is, and the purpose out of reading the
guide. The Getting help section points the user to some places where
they can get help, find a mentor, and introduce themselves.

@rust-highfive
Copy link
Collaborator

r? @Mark-Simulacrum

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 8, 2021
@henryboisdequin
Copy link
Contributor Author

@rustbot label +A-contributor-roadblock +C-enhancement +T-doc

@rustbot rustbot added A-contributor-roadblock Area: Makes things more difficult for new contributors to rust itself C-enhancement Category: An issue proposing an enhancement or a PR with one. A-docs Area: documentation for any part of the project, including the compiler, standard library, and tools labels Mar 8, 2021
@JohnTitor
Copy link
Member

I'm not sure if it's the best way to get it back and cloning means we have to update both, it's not ideal.
cc @rust-lang/wg-rustc-dev-guide

@henryboisdequin
Copy link
Contributor Author

I'm not sure if it's the best way to get it back and cloning means we have to update both, it's not ideal.
cc @rust-lang/wg-rustc-dev-guide

@JohnTitor I agree, would it be possible to create a tool to/make a bot automatically sync the file?

@JohnTitor
Copy link
Member

I'd update the rustc-dev-guide itself along with CONTRIBUTING.md not to make confusion rather than copying contents, I don't think duplicating is the right way here.

What is the rustc-dev-guide? What does rustc even mean?

I don't think it's a common case that the contributors do not know rustc. But we could put some brief explanations for rustc and rustc-dev-guide if needed.

@henryboisdequin
Copy link
Contributor Author

I'd update the rustc-dev-guide itself along with CONTRIBUTING.md not to make confusion rather than copying contents, I don't think duplicating is the right way here.

What is the rustc-dev-guide? What does rustc even mean?

I don't think it's a common case that the contributors do not know rustc. But we could put some brief explanations for rustc and rustc-dev-guide if needed.

Do you think a simplification/summary of https://rustc-dev-guide.rust-lang.org/contributing.html for the CONTRIBUTING.md would be a good option?

@joshtriplett
Copy link
Member

Thank you for the contribution, and for explaining your experiences as a new contributor!

While I do think it'd be nice to inline the contents here, I also think it'd be unfortunate to have the two get out of sync. And I don't think the improvement of inlining it is worth the likelihood of it getting out of sync.

However, we could improve the language used to link to the guide, to spell out exactly why that guide is what people will want to read; for instance, "the development guide for the Rust compiler" rather than "rustc-dev-guide". Would that help? Would you consider a PR to that effect? (I'd be happy to review it.)

Alternatively, if github can handle a symlink into a submodule and still present the contributor guide that way, we could do that. Otherwise, I think we should keep the current document but improve the wording.

@henryboisdequin
Copy link
Contributor Author

r? @joshtriplett

CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
@henryboisdequin henryboisdequin force-pushed the improve-contributing-md branch 2 times, most recently from ae83566 to dc36c43 Compare March 9, 2021 01:18
Fixes rust-lang#77215

As mentioned in rust-lang#77215, the current CONTRIBUTING.md links to the rustc-dev-guide.
Even though the rustc-dev-guide has lots of useful information for contributors,
one is already confused by reading the first line of the current CONTRIBUTING.md.

> To get started, read the [Getting Started] guide in the [rustc-dev-guide].

This line tells the contributor to go and read the rustc-dev-guide. What is
the rustc-dev-guide? What does rustc even mean? These are some of the
questions that went into my head when reading this line as a first time
contributor. By explaining what the rustc-dev-guide is and some platforms
to get help, a new contributor understands what the first step is and the process
is much clearer. The `About the [rustc-dev-guide]` section explains what
the rustc-dev-guide is, what rustc is, and the purpose out of reading the
guide. The `Getting help` section points the user to some places where
they can get help, find a mentor, and introduce themsevles.
CONTRIBUTING.md Outdated
@@ -1,9 +1,26 @@
# Contributing to Rust

Thank you for your interest in contributing to Rust!
Thank you for your interest in contributing to Rust! There are many ways to contribute
and we appreciate all of them.

To get started, read the [Contributing to Rust] chapter of the [rustc-dev-guide].
Copy link
Member

Choose a reason for hiding this comment

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

Instead of inlining part of the rustc-dev-guide, can we just rephrase this sentence to make it friendlier to new contributors? E.g., I'm thinking something like:

Suggested change
To get started, read the [Contributing to Rust] chapter of the [rustc-dev-guide].
Documentation for contributing to Rust is located in the [Guide to Rustc Development],
commonly known as the rustc-dev-guide. Despite the name, this guide documents
not just how to develop rustc (the Rust compiler), but also how to contribute to any part
of the Rust project.
To get started with contributing, please read the [Contributing to Rust] chapter of the guide.
That chapter explains how to get your development environment set up and how to get help.

Copy link
Member

Choose a reason for hiding this comment

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

I was actually suggesting that you remove the other changes in the PR and only make the first sentence friendlier (of course, you don't have to use my exact phrasing for the revised sentence, I was just giving a general idea of what I meant).

The other information is covered in the dev-guide pretty early on already IIRC.

@joshtriplett
Copy link
Member

This looks great. Thank you for working on it!

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Mar 9, 2021

📌 Commit bf40ac6 has been approved by joshtriplett

@bors bors 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 Mar 9, 2021
m-ou-se added a commit to m-ou-se/rust that referenced this pull request Mar 9, 2021
…g-md, r=joshtriplett

Update CONTRIBUTING.md

Fixes rust-lang#77215

As mentioned in rust-lang#77215, the current CONTRIBUTING.md links to the rustc-dev-guide.
Even though the rustc-dev-guide has lots of useful information for contributors,
one is already confused by reading the first line of the current CONTRIBUTING.md.

> To get started, read the [Getting Started] guide in the [rustc-dev-guide].

This line tells the contributor to go and read the rustc-dev-guide. What is
the rustc-dev-guide? What does rustc even mean? These are some of the
questions that went into my head when reading this line as a first-time
contributor. By explaining what the rustc-dev-guide is and some platforms
to get help, a new contributor understands what the first step is and the process
is much clearer. The `About the [rustc-dev-guide]` section explains what
the rustc-dev-guide is, what rustc is, and the purpose out of reading the
guide. The `Getting help` section points the user to some places where
they can get help, find a mentor, and introduce themselves.
bors added a commit to rust-lang-ci/rust that referenced this pull request Mar 9, 2021
Rollup of 8 pull requests

Successful merges:

 - rust-lang#81127 (Improve sift_down performance in BinaryHeap)
 - rust-lang#81879 (Added #[repr(transparent)] to core::cmp::Reverse)
 - rust-lang#82048 (or-patterns: disallow in `let` bindings)
 - rust-lang#82731 (Bump libc dependency of std to 0.2.88.)
 - rust-lang#82799 (Add regression test for rust-lang#75525)
 - rust-lang#82841 (Change x64 size checks to not apply to x32.)
 - rust-lang#82883 (Update Cargo)
 - rust-lang#82887 (Update CONTRIBUTING.md)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 74de9db into rust-lang:master Mar 9, 2021
@rustbot rustbot added this to the 1.52.0 milestone Mar 9, 2021
@henryboisdequin henryboisdequin deleted the improve-contributing-md branch March 9, 2021 13:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-contributor-roadblock Area: Makes things more difficult for new contributors to rust itself A-docs Area: documentation for any part of the project, including the compiler, standard library, and tools C-enhancement Category: An issue proposing an enhancement or a PR with one. 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.

Could CONTRIBUTING.md be improved?
10 participants