-
Notifications
You must be signed in to change notification settings - Fork 13k
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
Move contributing.md to rustc-dev-guide and point at getting started #73454
Conversation
(rust_highfive has picked a reviewer for you, use r? to override) |
@rustbot modify labels: +S-blocked |
The getting started guide is up: https://rustc-dev-guide.rust-lang.org/getting-started.html @rustbot modify labels: -S-blocked @nikomatsakis Would it make sense to do an FCP/MCP or something here? |
I don't really think it's necessary, but I also don't think it could hurt. I would happily second it. |
I'm going to go ahead and r+ -- if the MCP got 8 days into FCP, I think it's fine, and this is reversible. @bors r+ |
📌 Commit 3f6928f has been approved by |
@bors r- Well, wait, I guess that rust-lang/rustc-dev-guide#753 should merge first...? r=me in any case. |
@nikomatsakis I just merged #753 |
@bors r+ rollup |
📌 Commit 3f6928f has been approved by |
Move contributing.md to rustc-dev-guide and point at getting started See description on rust-lang/rustc-dev-guide#753
…arth Rollup of 8 pull requests Successful merges: - rust-lang#73454 (Move contributing.md to rustc-dev-guide and point at getting started) - rust-lang#73724 (Use WASM's saturating casts if they are available) - rust-lang#73726 (resolve: disallow labelled breaks/continues through closures/async blocks) - rust-lang#73753 (Use 'tcx for references to AccessLevels wherever possible.) - rust-lang#73781 (Update psm version) - rust-lang#73952 (Add option for local docker testing.) - rust-lang#73957 (disable BTree min_max test in Miri for now) - rust-lang#73975 (Document rustc_ast::ast::Pat) Failed merges: r? @ghost
See description on rust-lang/rustc-dev-guide#753