timonpost / cargo-toml-format Goto Github PK

Not sure if you're aware of, but a general purpose toml formatter exists. It's rather flexible and I use it for CI already. Might be worth a look :)

It is a bit misleading to have a main.rs with a fn main , which lets Rust believe the git repo can be installed as a binary, but it does nothing.

It shouldnt be too hard to fill in this function with a simple routine to load Cargo.toml from the current directory, apply whatever formatting routines are at alpha quality, and write it back out to disk or stdout.

E:\\programming\\ark\\ is referred to in the tests.

Hi Timon, big fan of your crates, had this idea:

1. In cargo_fmt_manifest.toml:

   workspace-only-dependencies = [
       "clap",
       "crossterm",
       "tokio",
   ]

2. This should now be an error:

   [dependencies]
   clap = "4.1"

   e.g.:

   Error: clap is defined at the workspace level, yet its version is specified as "4.1". Please specify it
   as clap = { workspace = true }. See:

   https://doc.rust-lang.org/cargo/reference/workspaces.html#the-dependencies-table

   Hmm, the error message should probably also mention the workspace-only-dependencies array.

What do you think?

Using stable:

error[E0423]: expected value, found crate `toml`
  --> tests/sandbox.rs:44:44
   |
44 |     let mut cargo = CargoToml::from_config(toml.to_string(), config).unwrap();
   |                                            ^^^^- help: use the path separator to refer to an item: `::`