-
Notifications
You must be signed in to change notification settings - Fork 888
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
Added Updated default version to Rustfmt docs config #4644
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Beautiful thank you! ❤️
FYI I tweaked the PR description to prevent closing the issue just yet. This change addresses the main issue so I wanted to go ahead and get it out ASAP, but there's a few other edge cases I'd like to tweak as well, for example ensuring the page default is something other than Line 101 in 19b34fe
Let me know if you'd be interested in working on that, or really anything else, as there's some other improvements I'd like to see in the config docs as well |
The doc page does not appear to honor the |
@jdahlstrom My bad. I'll try to figure out the cause and find a better way. |
Thanks for the report @jdahlstrom! |
backported in #4958 |
Fixes (mostly) #4633
Changes:
Updates the default version to the latest version released.