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

Make Rust 2018 primary and use "Edition Differences" for Rust 2015 #698

Closed

Conversation

mattheww
Copy link
Contributor

At present, some "Edition Differences" sections describe Rust 2018 additions to Rust 2015, some describe Rust 2015 limitations, and some are a mixture.

These commits attempt to make things consistent by putting the Rust 2018 description in the main text and information about Rust 2015 in the "Edition Differences" sections.

I've updated all the sections I could find, except for use-declarations.md
which needs more work (see #697).

Documents the Rust 2015 rules only in "Edition Differences" sections.
Documents the Rust 2015 rules in an "Edition Differences" section.
Documents the Rust 2015 rules in an "Edition Differences" section.
@mattheww
Copy link
Contributor Author

As discussed on Discord, I've prepared some different forms of this patch series.

  • 2019-10_rust2018_primary: has the 2018 edition in the main text (as in this PR), and the 2015 edition in "Edition differences"

  • 2019-10_rust2015_primary: has the 2015 edition in the main text, and the 2018 edition in "Edition differences"

  • 2019-10_common_primary: puts everything that varies between editions in "Edition differences", mostly leaving the main text describing what's true in both editions.

I've included changes to paths.md and use-declarations.md on those branches: those are some of the largest differences between the editions, so I think it makes sense to let people see what they might look like in the different arrangements.

But those commits are changing the text as well as rearranging it, so I don't think they're really part of this PR.

@ehuss
Copy link
Contributor

ehuss commented Nov 4, 2019

The intent was to have something like the "common primary" option, though there are some places where it was a little inconsistent. I think it will be difficult to have complete consistency, since some edition differences are large (like async/await, which just have brief callouts), so I suspect there will be a few special cases.

I'm not entirely happy with the "common" approach since it is a little disjoint. Some of the large callout blocks look a little strange to me. But the other approaches have their own drawbacks (such as making it less clear where the edition differences are).

If anything, I would vote for some of the "common" changes to make it more consistent. But I'd like to hear what other people think!

@joshtriplett
Copy link
Member

I like the idea of the reference always describing the current edition of Rust, and separately stating how past editions differ.

@mattheww
Copy link
Contributor Author

mattheww commented Sep 6, 2023

Closing: I don't think there's anything of value left here.

The Reference is now essentially in "Common primary" style (and this is documented in STYLE.md).

@mattheww mattheww closed this Sep 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants