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

add document describing rustfmt's stabilisation process #3486

Merged
merged 3 commits into from
May 9, 2019

Conversation

scampi
Copy link
Contributor

@scampi scampi commented Mar 31, 2019

Close #3197

no need to maintain a separate page for the options given the new configuration browser
@scampi
Copy link
Contributor Author

scampi commented Apr 21, 2019

@topecongiro what do you think about the proposed process ? Is it OK to be in a separate file or you'd prefer it in an existing one, e.g., Contributing.md ?

@topecongiro
Copy link
Contributor

@scampi Sorry about the delay, I have missed the notification 😞

Having this kind of documentation sounds good to me in general. Thank you for writing this up! W.r.t. stabilizing configuration option, however, I think we may first want to consider whether the current interface and design is good enough for stabilization. Could refer to it in this documentation?

@scampi
Copy link
Contributor Author

scampi commented May 8, 2019

@topecongiro I have added a condition taken from your comment.

@topecongiro topecongiro merged commit a2264cb into rust-lang:master May 9, 2019
@topecongiro
Copy link
Contributor

@scampi Thank you!

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.

Process for config option stabilisation
2 participants