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 custom rustfmt.toml #594

Merged
merged 1 commit into from
Dec 12, 2022
Merged

Add custom rustfmt.toml #594

merged 1 commit into from
Dec 12, 2022

Conversation

bobbinth
Copy link
Contributor

@bobbinth bobbinth commented Dec 10, 2022

This PR adds custom source formatting rules as discussed in #456. I ended up excluding some of the ones we discussed there because the resulting formatting looked a bit odd to me, but most of the rules are included.

Update: I turned off more custom rules because CI complained that some of them are available only on nightly.

Copy link
Contributor

@vlopes11 vlopes11 left a comment

Choose a reason for hiding this comment

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

Looks good! We just forgot to add wrap_comments = true, but it is also not stable yet

@itzmeanjan
Copy link
Contributor

Looks good! We just forgot to add wrap_comments = true, but it is also not stable yet

It'll be good to have this.

@bobbinth bobbinth merged commit be833a6 into next Dec 12, 2022
@bobbinth bobbinth deleted the bobbin-rustfmt-update branch December 12, 2022 14:35
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.

3 participants