Skip to content

Cargo.tomlをフォーマットする #469

@qryxip

Description

@qryxip

内容

Cargo.tomlのフォーマットには、Rustとして公式にconventionがあります。
Cargo.toml conventions

また公式のconventionには書かれていない慣習として、「dependencyのクレートはアルファベット順に並べる」というものがあります。今から2年半程前に旧cargo-add.exeに「dependencyがアルファベット順になっているのならそれを保ったまま挿入する」という挙動が追加され、その後この挙動のままCargo本体に統合されたことでアルファベット順にするという慣習は広まったように思えます。
killercup/cargo-edit#421
(ただしそれ以前にもアルファベット順にしていたところは結構あったような記憶があります)

現在、上記のconvention(dependencyのアルファベット順も含む)にのっとったフォーマッタの実装がRustfmtに入ろうとしています。
rust-lang/rustfmt#5240
これを使って一度Cargo.tomlをフォーマットします。

Pros 良くなる点

  • Cargo.tomlの書き方に悩まずにすむ

Cons 悪くなる点

実現方法

rust-lang/rustfmt#5240のRustfmtでCargo.tomlをフォーマットする。

ただし一部変なフォーマットをするようなので、そこは無視して元のままにする。

-[target."cfg(windows)".dependencies]
+[target."cfg(windows)" .dependencies]
 humansize = "2.1.2"

フォーマッタの実装はnightlyですらないレビュー待ちPRの状態であるため、CIで動作させるのは控える。

VOICEVOXのバージョン

N/A

OSの種類/ディストリ/バージョン

  • Windows
  • macOS
  • Linux

その他

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions