refactor: simplify AddPackageSignerParams merge field from Option<bool> to bool

This commit is contained in:
Aiden McClelland
2026-03-09 23:59:58 -06:00
parent 2586f841b8
commit 73c6696873

View File

@@ -59,8 +59,7 @@ pub struct AddPackageSignerParams {
#[ts(type = "string | null")] #[ts(type = "string | null")]
pub versions: Option<VersionRange>, pub versions: Option<VersionRange>,
#[arg(long, help = "help.arg.merge")] #[arg(long, help = "help.arg.merge")]
#[ts(optional)] pub merge: bool,
pub merge: Option<bool>,
} }
pub async fn add_package_signer( pub async fn add_package_signer(
@@ -89,7 +88,7 @@ pub async fn add_package_signer(
.as_authorized_mut() .as_authorized_mut()
.upsert(&signer, || Ok(VersionRange::None))? .upsert(&signer, || Ok(VersionRange::None))?
.mutate(|existing| { .mutate(|existing| {
*existing = if merge.unwrap_or(false) { *existing = if merge {
VersionRange::or(existing.clone(), versions) VersionRange::or(existing.clone(), versions)
} else { } else {
versions versions