search

Electron100 / butane

An ORM for Rust with a focus on simplicity and on writing Rust, not SQL
Apache License 2.0
91 stars 13 forks source link

Alpha sort keys in Cargo.toml files #40

Closed jayvdb closed 1 year ago

jayvdb commented 1 year ago

Followup to https://github.com/Electron100/butane/pull/39, this uses reorder_keys. No worries if you reject because it also changes the package metadata sections. If rejected, I'll see if I can find another tool which can reorder only the dependencies.

Electron100 commented 1 year ago

Hmm, I'm going to have to think about this one a little bit. The ordering of dependencies is nice (I've been trying to keep them ordered manually, but obviously I haven't fully succeeded), but I don't love the re-ordering of other sections.

jayvdb commented 1 year ago

Looks like taplo can be used per https://github.com/tamasfe/taplo/issues/382#issuecomment-1444008846 .

I'll update this PR after https://github.com/Electron100/butane/pull/52 & https://github.com/Electron100/butane/pull/51 are merged.

jayvdb commented 1 year ago

I've updated this. I havent built CI to enforce it yet - there are only a few contributors atm, so not a high priority atm.