Closed chriskrycho closed 6 years ago
Also needs documenting: #[cfg(target_feature = "")]
. I'll get around to it if I don't forget.
Sorry it took me so long to respond, @mattico—thanks for the note! I'll flag it as something to do when adding the other similar configurable compilation bits with e.g. #0131.
Would it be useful to have a link in the nightly lang reference to this issue?
@mrhota that's a great idea!
Note for anyone viewing this page there is a long list of RFC that still need the documentation status evaluated. I am happy to merge changes from forks of the gist.
Even simpler, let's have interested people fork my new copy of the original (for which a thousand thanks, @Eh2406!) and I can more easily merge them and track them myself!
Update to above comment, @mattico: target_feature
will be covered in #1199!
Could you add something in the description for those of us who want to help but don't know what you want us to do? Right now all I see is a big list of stuff but I'm not sure what you want us to help with and how exactly you want us to go about it. This is a very monolithic issue and it's hard to know what to do if you haven't helped with this kind of thing before.
Absolutely! I'll add that later this afternoon.
Note: This is integrated into the body of the issue as well; just adding it here so everyone who's watching the issue sees it via notification!
For right now, since you can't submit updates to the issue directly, there are two major ways you can help:
Write documentation. For the RFCs already triaged and filed under the major section C. Documentation Needed and specifically the subsection 2. Write reference material for undocumented, implemented, stabilized RFC features below, you can open a pull request and reference this issue in its description. I'll keep an eye out for inbound links here and add those links to the relevant items, and once they're merged, we'll mark that item as done.
Triage issues. Work the gist where I'm tracking the triaged-state, specifically looking at the Not Reviewed at All and Stabilized, Not Reviewed sections and evaluating whether they're documented or not. You can fork the gist and I will keep an eye out, checking roughly daily, for any changes to merge back into that document. (Do take a look at what others may have forked it so you don't duplicate work needlessly!) I'll then pull those over into this master issue.
When working through those, if you determine that they're already documented, please leave a note (and link!) on your copy of the gist as to where so I can link it in this document. That dramatically decreases the time it takes me to confirm it—I double-check all of these.
@chriskrycho You can tick C.1 (or convert all the sub-bullets to boxes and tick them all!). See #35.
@chriskrycho I opened a pull request for #558.
@mrhota outstanding! I'll make that tweak in a bit. (There will be more to add, but it's an awesome start!)
@ekse 💥
@chriskrycho I think that I've got all of the simple RFCs done in the gist now, and have categories and links for just about everything else to help anyone wanting to check the more complicated RFCs more carefully.
Excellent! I'll have a nice block of time off and will review those quickly and hit some of the more complicated ones starting next week!
rust-lang/rfcs#1869 can be ticked off as well.
@chriskrycho OK, you can check off C.2 UFCS discussion: see #45
loop_break_value
is done@chriskrycho another one! merged #44, so you can check off C.2 integer overflow. Also, remember to check off C.2 Unambiguous function call syntax since we resolved that with #45
I checked off integer overflow, ufcs, loop/break, and windows subsystem.
Thanks @brson @mrhota @dhardy @sanmai-NL!
My vacation was far more productive than I planned in non-Rust categories, in that we bought a house; but accordingly less productive here. Current goal is to spend at least one hour every week working on this going forward. Here's hoping. :wink:
Looks like the tracking list hasn't been updated in a while, it's missing the following RFCs:
Thank you very much @lukaramu!
And status update for everyone else: I ended up in a surprising spot as maintainer of ember-cli-typescript and that ate my summer. I'm close to getting that stable and coming back to this.
I'm confused both as to what the statuses are (what do stabilized and reviewed mean?) and where the current work is.
Totally fair! Stabilized means the RFC feature has been merged to stable Rust. Reviewed means I or someone else had looked over the documentation and confirmed whether an RFC feature is documented or not.
Status is basically the above comments and the Gists linked in them. I am actually planning on taking some time on Jan. 1 to write up some comments on all of this and expect to knock out the rest of the review triage in the next month or so, which should make it way easier for people to just document those currently-undocumented RFC features.
Any progress? I'd like to start trying to submit PRs for easier RFCs.
No, and sorry for the delay. I will pick this work up Tuesday afternoon!
Is there anything I can do to help you out at this point?
Can you unpack my house? 😜
I’ll be at it in the next week or so. Apologies for the several delays. It’s one of my main goals for Rust stuff I do this year, so I will get to it. Holidays plus moving plus cancer in the extended family has made for an especially busy crunch.
Unfortunately I'm on the wrong continent at the moment!
I'm sorry to hear about your family, I hope things go well. I look forward to getting things sorted. When you do, would it perhaps make sense to file a bug for each RFC?
People can at any time fined a RFC that is not documented and make a pr to improve the documentation. That is not dependent on this organisation getting updated.
@alercah just a thought: could you provide a snapshot or summary of where this issue currently is, accounting for all comments and stuff? That would probably take some pressure off @chriskrycho since he could then just copy your snapshot/summary into the OP when he gets a free moment.
Come to think of it, if you want to work on it together, I'm going to get on IRC in the #rust-docs channel. I recently wanted to summarize this issue myself, and I'm feeling inspired now.
ok, @chriskrycho, I'm updating your gist in my fork here. Hopefully it'll be easy to incorporate the updates into your copy. I'm also incorporating changes from other forks of your gist, so you'll just be able to copy over changes from mine.
Excellent!
I've added a new project to help track things more easily, and am moving data from @mrhota's Gist over to it. Proposed instructions to use are in #220.
I've now put all the "needs review" RFCs in, except for ones which I poked at and they completely obviously didn't (either not stable or not related to the language).
Thank you very much!
I removed all the unstable RFCs from the Needs Review column, since they'll get docs as part of their work. A few I moved into the Awaiting column, and a few I closed as unrelated since they were obviously e.g. library-only PRs.
GitHub needs a "You're my hero!" response-moji.
I've gone through and added all the ones marked as needing documentation, up to but not including the "Macros" section on the Gist. Only a few left.
I've opened a bigger issue for macros, and linked the relevant RFCs, since macro documentation is sorely lacking. The remaining ones in "It's complicated..." are either library-only or documented to the extent that they need to be in the reference, so no need to do further work. I would say the Gist is now fully ported into the issue tracker & project page. I propose closing this issue.
👏 Thank you so much for picking up where I left off, not least b/c I never got back to it. 😭 I'm in favor of closing if everyone else working on this is satisfied that it is indeed full ported into the issue tracker!
This is awesome! I agree, looks like you've gotten everything. Let's close.
Overview
Tracking issue for RFC #1636. Let's make the reference a reliable resource for all Rustaceans! :tada: This might seem intimidating, but the reality is that if we just chip away at it together, we'll be done in no time!
I'm going to be updating this parent issue with a master list of items that need to be documented in the reference as I find them. Quoting the RFC text:
Note that step 1 should be fairly straightforward; the main issue will be assembling the list of accepted-and-implemented-but-undocumented RFCs. (Also, any RFCs accepted before RFC 1636 but not yet stabilized should presumably be documented under the rules it establishes, but if I'm wrong about that, someone should let me know and I'll include them as well.)
Also, a pre-emptive apology for the scale of this issue description. We have let things get into a rough spot. (I plan to create documentation issues for each of the required items below once this list is completed, so this thread doesn't become completely unmanageable.)
Status: Last updated May 13, 2017, now processing a copy of @Eh2406's wonderful gist based on @matthewjasper's fork here.
"How Can I Help?"
For right now, since you can't submit updates to the issue directly, there are two major ways you can help:
Write documentation. For the RFCs already triaged and filed under the major section C. Documentation Needed and specifically the subsection 2. Write reference material for undocumented, implemented, stabilized RFC features below, you can open a pull request and reference this issue in its description. I'll keep an eye out for inbound links here and add those links to the relevant items, and once they're merged, we'll mark that item as done.
Triage issues. Work the gist where I'm tracking the triaged-state, specifically looking at the Not Reviewed at All and Stabilized, Not Reviewed sections and evaluating whether they're documented or not. You can fork the gist and I will keep an eye out, checking roughly daily, for any changes to merge back into that document. (Do take a look at what others may have forked it so you don't duplicate work needlessly!) I'll then pull those over into this master issue.
When working through those, if you determine that they're already documented, please leave a note (and link!) on your copy of the gist as to where so I can link it in this document. That dramatically decreases the time it takes me to confirm it—I double-check all of these.
A. Tracking
(This section will go away entirely once all of the RFCs have been flagged for documenting or marked documentation-not-needed here.)
RFCs reviewed
Currently: 115/301
RFCs unreviewed
B. Status unclear
Some of these are still in-flight; and some of them are just the kind of thing that I don't even fully grok yet well enough to see if they're documented. For these, unchecked means "status unknown"; checked means "status known and added to the latter bits appropriately."
C. Documentation needed
0. Accepted, not-yet-stabilized, undocumented RFCs
0.1. Document implemented, unstable RFCs
These should be considered the highest priority for documentation, as these are issues which fall under the rest of the rules of [RFC #1636], in that they need to be documented before stabilization. (That will presumably just happen before stabilizing as usual, but I'm including them here for completeness.)
plugin-registrar
APIrepr_align
– implementation is in-progress, not yet landed on nightly0.2. Track accepted, not-yet-implemented or implementation-in-progress, undocumented RFCs
These will eventually require documentation, but as they aren't even (fully) implemented yet, there is no urgency here.
{Vec,String}::splice
– tracking issuemacro_rules!
– tracking issuemacro_rules
– tracking issueshared_from_slice
– tracking issueManuallyDrop
to core – tracking issueeprint!
andeprintln!
– tracking issue1. List accepted, implemented, already-stabilized, undocumented RFCs
This list can be added directly to a newly(-to-be)-created appendix to the Reference.
[x] Create the appendix (once finished, this list should do just fine, perhaps with some massaging for descriptiveness)
libstd-facade
– there is one reference to the facade, in 6.3.13 Compiler Features under a discussion of#[no_std]
, but no explanation of its meaning. Nor, as far as I can tell, do the relevant sections of the standard library documentation explain this.Self
), but others aren't at all: coherence and orphan rules are covered nowhere. (Currently, the writeup here seems to be the best source on coherence?)crt_link
unaligned_access
2. Write reference material for undocumented, implemented, stabilized RFC features
Each of the features listed above in (1) needs to be documented more formally in the reference.
&mut
patterns – symmetry of pattern syntax not documented3. Update out-of-date/incomplete sections of the reference
[ ] List of language items
[ ] Coherence
[ ] Orphan rules (#5)
[ ] Lifetime elision
D. Documentation not needed
0. Already documented
1. Retired
These items were accepted, but never implemented and not currently *planned* to be implemented and therefore not in need of documentation.
- [#0008]: New intrinsics - [#0112]: Remove cross borrowing coercion. Replaced by the removal in [#0139]. - [#0115]: do not fall back to `int` if cannot infer integer; replaced with [#0212]. - [#0155]: only allow "anonymous impl" (`impl Foo`) in the same module as the data type is defined (`struct Foo` or `mod Foo`). Replaced by [#0735].2. Removals
Some items constitute not *additions to be documented* but *things removed* from the language. These do not require documentation (for obvious reasons!).
- [#0026]: Remove the `priv` keyword - [#0059]: Remove `~` in favor of `box` and `Box` - [#0063]: Tighten restrictions on `mod` - [#0093]: Remove localization from `format!` - [#0139]: Remove cross borrowing entirely. - [#0230]: Remove runtime - [#0256]: Remove refcounting `Gc3. Parser-specific
Some changes are specific to the parsing (though they affect the language). These should be documented not in the reference but in the language grammar.
- [#0092]: struct grammar change/simplification (may also be touched in #13) - [#0342]: keywords, listed in the grammar and linked explicitly at [2.2 String table productions][2.2]4. Process and conventions
5. Non-language features
Note: I've just copied over the original issue as it was—wholesale. Note that some of the links here referencing things as "already documented" will degrade given the new structure of the book.