-
Notifications
You must be signed in to change notification settings - Fork 38
Add bevy fmt. #670
Copy link
Copy link
Open
Labels
A-CLIRelated to the main CLI and not a more specific subcommandRelated to the main CLI and not a more specific subcommandC-FeatureMake something new possibleMake something new possibleD-ModestA "normal" level of difficulty; suitable for simple features or challenging fixesA "normal" level of difficulty; suitable for simple features or challenging fixesX-ControversialThere is active debate or serious implications around merging this PRThere is active debate or serious implications around merging this PR
Metadata
Metadata
Assignees
Labels
A-CLIRelated to the main CLI and not a more specific subcommandRelated to the main CLI and not a more specific subcommandC-FeatureMake something new possibleMake something new possibleD-ModestA "normal" level of difficulty; suitable for simple features or challenging fixesA "normal" level of difficulty; suitable for simple features or challenging fixesX-ControversialThere is active debate or serious implications around merging this PRThere is active debate or serious implications around merging this PR
What problem does this solve or what need does it fill?
cargo fmtdoes not format all rust files in src. It instead formats all modules it can find. If the modules are imported from a macro, then it simply will not format them. Modules imported by a macro can allow for some very powerful techniques that I believe would benefit bevy.What solution would you like?
A
bevy fmtwould run rustfmt individually on each rust file in src.What alternative(s) have you considered?
I have my own
cargo please_format, but this behaviour seems useful to more than just me.Additional context
Some discussion in the discord.