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

chore: Run rustfmt #256

Closed
wants to merge 1 commit into from
Closed

chore: Run rustfmt #256

wants to merge 1 commit into from

Conversation

Hywan
Copy link
Contributor

@Hywan Hywan commented Jul 5, 2021

Description

This patch is the result of running rustfmt **/*.rs from the root of
this project. The goal is the following: When a user (like me) edits a
file, the file is automatically formatted on save; which results in
larger diffs than needed. Let's fix the coding styles once and for all
thanks to rustfmt :-)!

This patch is the result of running `rustfmt **/*.rs` from the root of
this project. The goal is the following: When a user (like me) edits a
file, the file is automatically formatted on save; which results in
larger diffs than needed. Let's fix the coding styles once and for all
thanks to `rustfmt` :-)!
@TheDan64
Copy link
Owner

TheDan64 commented Jul 5, 2021

Hi, thanks for the PR. I appreciate the effort but unfortunately it will not be accepted. We already have an open issue here: #75 which documents that we're waiting for rustfmt 2.0 so that we can use unstable rustfmt features rather than requiring users to have nightly just to format inkwell.

Regarding formatting on save, editors usually allow you to disable format on save on a per-project basis

@TheDan64 TheDan64 closed this Jul 5, 2021
@Hywan
Copy link
Contributor Author

Hywan commented Jul 6, 2021

Yeah, OK. I'm not sure to understand why you don't want to get a clean code base with the current rustfmt version, which will format 99% of the code as you expect, but that's your choice :-).

@TheDan64
Copy link
Owner

TheDan64 commented Jul 6, 2021

It should be out relatively soon I hope. There's only one issue left: https://github.com/rust-lang/rustfmt/milestone/8

@Hywan
Copy link
Contributor Author

Hywan commented Jul 6, 2021

Good to know :-)!

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.

2 participants