Experiment/cargo.toml formatting

[s373r]

Checklist

• Add CI step(s)
• Wait for changes from the IoC branch
  • Resource listing with patterns + Wide integration of ResourceRef/ResourceSelector #1685

[sergiimk]

I like it! 👍

The line break in empty arrays was the only thing that jumped at me:

default = [
]

but I'm ready to live with it.

[sergiimk]

Can we introduce a new make fmt target? It's a bit strange to mix clippy and code formatting together.

And if we keep this target - we probably should format code before running clippy not to rebuild twice in case of formatting changes.

[s373r]

It's a bit strange to mix clippy and code formatting together

IMHO, Clippy's rules, which specifically automaticly change the code, are perfectly suited for lint-fix

Can we introduce a new make fmt target?

But you're right -- it's better to have a make target for that

--

Will do

[s373r]

Done

[sergiimk]

Does cargo-sort respect these comments as group separators or you sorted these manually?

[s373r]

Does cargo-sort respect these comments as group separators or you sorted these manually?

That is cargo-sort.

The only thing I did manually was to add the groups in the comments so that sorting wouldn't break up the grouping within Infra, e.g.:

- # Infra
- ## Outbox
+ # Infra (core)
+ # Infra - Outbox