-
Notifications
You must be signed in to change notification settings - Fork 515
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
Overview Chapter Followup #674
Comments
cc @rust-lang/wg-rustc-dev-guide |
With some items added to rust-lang#674 Co-Authored-By: Chris Simpkins <[email protected]>
With some items added to #674 Co-Authored-By: Chris Simpkins <[email protected]>
This goal is surprising to me, why prose? I find bullets easier to read.
You don't need to catalogue all the necessary information ahead of time (you can have unknown knowns). For example lints wouldn't need to state ahead of time whether they needed typeck to run or not: https://discord.com/channels/442252698964721669/443151243398086667/724028959209291817
I'm interested in helping with this. |
I also find bullets more easy... prose can be intimidating. |
That's helpful feedback. I think the original motivation was that the bullet points in this chapter already contain paragraphs, so prose may be a bit more natural...
Awesome! Thanks :) So IIRC, that was referring to this para: https://rustc-dev-guide.rust-lang.org/overview.html#bootstrapping Basically, there was a feeling that it moves too fast and wouldn't make sense to new comers. Any thoughts? |
This is really confusing. It would be better phrased as
In general I think it would be better to have a separate page for bootstrap and replace this section with a link to that page. Otherwise there's too much in too little time. I have more thoughts but I think they would be better placed in a PR making these changes directly :P |
Oh hmm I see there's a separate |
I'm interested in helping with this, I had to mess with queries a lot for rust-lang/rust#73566. |
@jyn514 That would be great! |
this is pinned... is it current, should it remain open |
Triage: unpinning this because it's kinda overly generic and not very actionable; we should prefer opening specific tracking issues. |
This issue exists to track things we want to improve on. Please add things here:
The text was updated successfully, but these errors were encountered: