Add default settings for rustfmt and run on applications
#1090
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is addressing Issue #529
As development on Theseus continues the importance of a "no-thinking-required" format increases. This is where
rustfmt
comes in. Based off previous discussion we will not use the defaultrustfmt
but would use a customized version for the project using therustfmt.toml
file. Some concerns on the formatting is how it will handle vertical line spacing, chaining, and others.This PR does 2 things:
rustfmt.toml
file which are easily customizable.cargo fmt
onapplications
directory to demonstrate what these values will create.Some changes to the formatting based off the new default values include:
And more not listed
Because of how customizable
rustfmt.toml
is it should be not too difficult to modify the settings further to haverustfmt
leave do more or less when run. If this PR is approved then more folders can havecargo fmt
run on them handling each case by case until they are all no longer being ignored.