Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Unify Cargo.toml format #1170

Closed
wants to merge 5 commits into from
Closed

Unify Cargo.toml format #1170

wants to merge 5 commits into from

Conversation

koushiro
Copy link
Contributor

@koushiro koushiro commented Apr 13, 2022

  • Add taplo config for unifying Cargo.toml format
  • Add CI job for checking Cargo.toml format

@koushiro koushiro requested review from a team and chevdor as code owners April 13, 2022 09:49
Signed-off-by: koushiro <[email protected]>
@gilescope
Copy link
Contributor

FYI I think we are seeing some movement on cargo fmt doing Cargo.tomls finally: rust-lang/rustfmt#5240

@bkchr
Copy link
Member

bkchr commented Apr 13, 2022

Ty @koushiro, but for now we will not use any external tool to format the toml files. Maybe when it is in rustfmt and stable, we can again speak about this.

@bkchr bkchr closed this Apr 13, 2022
@nuke-web3
Copy link
Contributor

#1050 FYI was my take at a uniform convention, not automated though

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

Successfully merging this pull request may close these issues.

4 participants