-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Move contributing.md to rustc-dev-guide and point at getting started #73454
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
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