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

Unify Cargo.toml format#1170

Closed
koushiro wants to merge 5 commits intoparitytech:masterfrom
koushiro:unify-toml-format
Closed

Unify Cargo.toml format#1170
koushiro wants to merge 5 commits intoparitytech:masterfrom
koushiro:unify-toml-format

Conversation

@koushiro
Copy link
Copy Markdown
Contributor

@koushiro koushiro commented Apr 13, 2022

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

Signed-off-by: koushiro <koushiro.cqx@gmail.com>
Signed-off-by: koushiro <koushiro.cqx@gmail.com>
Signed-off-by: koushiro <koushiro.cqx@gmail.com>
Signed-off-by: koushiro <koushiro.cqx@gmail.com>
@koushiro koushiro requested review from a team and chevdor as code owners April 13, 2022 09:49
Signed-off-by: koushiro <koushiro.cqx@gmail.com>
@gilescope
Copy link
Copy Markdown
Contributor

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

@bkchr
Copy link
Copy Markdown
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
Copy Markdown
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