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

Move style guide #3320

Merged
merged 1 commit into from
Oct 23, 2022
Merged

Conversation

joshtriplett
Copy link
Member

@joshtriplett joshtriplett commented Sep 24, 2022

RFC 3309 moved the style guide to rust-lang/rust. In addition, the
link from
RFC 2436 to
the style guide does not seem to have ever worked, and the style guide
has never been available anywhere in rendered form.

Update RFC 2436 minimally to point to the rendered style guide.

Rendered

@joshtriplett joshtriplett added not-rfc For PRs that fix things like spelling mistakes, wrong file names, etc. T-style Relevant to the style team, which will review and decide on the RFC. labels Sep 24, 2022
@joshtriplett
Copy link
Member Author

Currently draft, until rust-lang/rust#102239 gets merged.

@calebcartwright
Copy link
Member

I think we should definitely get rid of the specifics here as that's been an ongoing source of confusion. One of the things I heard when I first started contributing to rustfmt was that the rustfmt team of that time was entirely unaware that this rfc version of the style guide specifics existed here.

@calebcartwright
Copy link
Member

I've a couple questions more related to the execution of moving the guide into r-l/rust that I'll ask here:

  • What's the plan for the repo where the guide is currently maintained? I feel like it should be functionally archived to avoid any confusion or bifurcation (though I don't have access to do this myself)
  • Any thoughts on how we could potentially lower the frequency of revisiting topics that were previously discussed at length and resolved (e.g. are there any steps we should take to capture/point to past discussions to hopefully reduce the odds of r-l/rust getting spammed with "switch default from spaces to tabs" types of issues?)
    • Perhaps something as simple as creating a small list of quick links to those past discussions to capture the frequently asked/requested items?
  • Should we run a final triage pass over open issues/PRs in the current repo before migrating to see if there's anything that can be closed out and pulled in as part of the move, or should we move first and then recreate as needed?

@joshtriplett
Copy link
Member Author

joshtriplett commented Oct 9, 2022

@calebcartwright Regarding your first and third points, RFC 3309 mentions:

This RFC proposes to move the Rust style guide to the rust-lang/rust repository, rather than its current location in the RFCs repository. Style work may additionally take place in the fmt-rfcs repository (which this RFC proposes to revive). Larger style proposals may wish to start there rather than as PRs to the style guide. The style team may choose to change this process.

I think we should keep fmt-rfcs for now, and decide as a team if we want to continue using it or use some other process for larger pieces of style work.

@joshtriplett
Copy link
Member Author

joshtriplett commented Oct 9, 2022

@calebcartwright wrote:

  • Any thoughts on how we could potentially lower the frequency of revisiting topics that were previously discussed at length and resolved (e.g. are there any steps we should take to capture/point to past discussions to hopefully reduce the odds of r-l/rust getting spammed with "switch default from spaces to tabs" types of issues?)

    • Perhaps something as simple as creating a small list of quick links to those past discussions to capture the frequently asked/requested items?

We've talked about having a similar FAQ for other frequently requested items, such as "make braces optional on if and other block constructs". I think we could add a few style items to that FAQ, and if the list of style items in that FAQ gets too long we could split out a style-specific FAQ and link to it.

RFC 3309 moved the style guide to `rust-lang/rust`. In addition, the
link from
[RFC 2436](https://rust-lang.github.io/rfcs/2436-style-guide.html) to
the style guide does not seem to have ever worked, and the style guide
has never been available anywhere in rendered form.

Update RFC 2436 minimally to point to the rendered style guide.
@joshtriplett joshtriplett marked this pull request as ready for review October 16, 2022 19:47
@joshtriplett
Copy link
Member Author

rust-lang/rust#102239 has been merged, so this can now be merged as well.

@joshtriplett joshtriplett merged commit d6b5c97 into rust-lang:master Oct 23, 2022
@joshtriplett joshtriplett deleted the sartorial-switch branch October 23, 2022 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
not-rfc For PRs that fix things like spelling mistakes, wrong file names, etc. T-style Relevant to the style team, which will review and decide on the RFC.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants