Auto merge of #13748 - epage:kebab, r=weihanglo

refactor(config): Consistently use kebab-case

This shouldn't change the behavior but makes it safer if
- We add new fields where it will matter
- Copy/paste these for new structs

I did not change things related to the Index because we are already stuck with that case (whether we want it or not)

Came across this when working on #13540 and almost made the mistake of copying what was already there
This commit is contained in:
bors 2024-04-13 13:04:42 +00:00
commit 8dd6db4f12
4 changed files with 7 additions and 1 deletions

View File

@ -131,6 +131,7 @@ impl NewOptions {
}
#[derive(Deserialize)]
#[serde(rename_all = "kebab-case")]
struct CargoNewConfig {
#[deprecated = "cargo-new no longer supports adding the authors field"]
#[allow(dead_code)]

View File

@ -70,6 +70,7 @@ pub struct DirectorySource<'gctx> {
/// The file name is simply `.cargo-checksum.json`. The checksum algorithm as
/// of now is SHA256.
#[derive(Deserialize)]
#[serde(rename_all = "kebab-case")]
struct Checksum {
/// Checksum of the package. Normally it is computed from the `.crate` file.
package: Option<String>,

View File

@ -227,6 +227,7 @@ pub const CRATES_IO_DOMAIN: &str = "crates.io";
/// The content inside `.cargo-ok`.
/// See [`RegistrySource::unpack_package`] for more.
#[derive(Deserialize, Serialize)]
#[serde(rename_all = "kebab-case")]
struct LockMetadata {
/// The version of `.cargo-ok` file
v: u32,

View File

@ -2514,6 +2514,7 @@ impl<'de> Deserialize<'de> for SslVersionConfig {
}
#[derive(Clone, Debug, Deserialize, PartialEq)]
#[serde(rename_all = "kebab-case")]
pub struct SslVersionConfigRange {
pub min: Option<String>,
pub max: Option<String>,
@ -2644,6 +2645,7 @@ impl BuildTargetConfig {
}
#[derive(Deserialize, Default)]
#[serde(rename_all = "kebab-case")]
pub struct TermConfig {
pub verbose: Option<bool>,
pub quiet: Option<bool>,
@ -2656,13 +2658,14 @@ pub struct TermConfig {
}
#[derive(Debug, Default, Deserialize)]
#[serde(rename_all = "kebab-case")]
pub struct ProgressConfig {
pub when: ProgressWhen,
pub width: Option<usize>,
}
#[derive(Debug, Default, Deserialize)]
#[serde(rename_all = "lowercase")]
#[serde(rename_all = "kebab-case")]
pub enum ProgressWhen {
#[default]
Auto,