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

Fixed Updated default version to Rustfmt docs config #4662

Merged
merged 1 commit into from
Jan 22, 2021

Conversation

gunadhya
Copy link
Contributor

Fixes (mostly) #4633
Changes:
Updates the default version to the latest version released.
Also fixes the issue raised at #4644 (comment)
Sry about that.

Signed-off-by: gunadhya <6939749+gunadhya@users.noreply.github.com>
@gunadhya
Copy link
Contributor Author

Reply to #4644 (comment)
@calebcartwright I'd be happy to work on any other improvements in the config.

Copy link
Member

@calebcartwright calebcartwright left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the quick fix! Change looks good and seems to address the query param issue 👍

@calebcartwright calebcartwright merged commit 79c3696 into rust-lang:master Jan 22, 2021
@calebcartwright
Copy link
Member

Reply to #4644 (comment)
@calebcartwright I'd be happy to work on any other improvements in the config.

Great! Have opened a couple new issues for some of them, see #4664 and #4665, lmk if either interest you

@karyon
Copy link
Contributor

karyon commented Oct 26, 2021

backported in #4958

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants