-
Notifications
You must be signed in to change notification settings - Fork 504
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
modules: describe both module filename styles without giving a clear preference #1703
base: master
Are you sure you want to change the base?
Conversation
I think changing the expressed preference would require a consensus to change it. There was an intentional decision in 2018 to try to migrate towards the Rust 2018 style. |
I didn't realize that this was an intentional decision. There's little other indication of a goal of pushing the ecosystem towards the 2018 style. Not even the compiler itself was ever systematically adjusted. So, that would be a t-lang FCP then? |
@rustbot labels +I-lang-nominated |
@RalfJung IIRC, in 2018 when we designed the new module system, it was at that time an intentional decision to pointedly present it as "the new way" by contrast with "the old way". We didn't particularly push people to switch. I'm not suggesting that we would necessarily object to this proposal, but I do think it's fair to say that this proposal represents a change in recommendations, and it probably warrants at least a discussion. |
As some archaeology, this went in with: At the time, @ehuss said:
The tracking issue, where the FCP happened, was: On that thread, Josh said:
There's also some discussion between then-lang members suggesting that deprecation would have run into blocking objections. The RFC was: What that RFC says specifically is:
|
There's currently no clear consensus on which filename style to use for modules, and many people have a style tha mixes both options. So let's make the reference just describe the facts, without saying which variant is preferred.
Also see the discussion on IRLO.