Merge pull request #572 from epage/perf

perf: Reduce owners overhead

src/steps/owner.rs

@@ -119,7 +119,7 @@ impl OwnerStep {
 
 pub fn ensure_owners(pkgs: &[plan::PackageRelease], dry_run: bool) -> Result<(), CliError> {
     for pkg in pkgs {
-        if !pkg.config.publish() {
+        if !pkg.config.publish() || !pkg.ensure_owners {
             continue;
         }
 

src/steps/plan.rs

@@ -85,6 +85,8 @@ pub struct PackageRelease {
 
     pub planned_version: Option<Version>,
     pub planned_tag: Option<String>,
+
+    pub ensure_owners: bool,
 }
 
 impl PackageRelease {
@@ -132,6 +134,7 @@ impl PackageRelease {
 
         let planned_version = None;
         let planned_tag = None;
+        let ensure_owners = config.publish() && !config.owners().is_empty();
 
         let pkg = PackageRelease {
             meta: pkg_meta.clone(),
@@ -151,6 +154,7 @@ impl PackageRelease {
 
             planned_version,
             planned_tag,
+            ensure_owners,
         };
         Ok(pkg)
     }

src/steps/release.rs

@@ -63,6 +63,10 @@ impl ReleaseStep {
             if let Some(level_or_version) = &self.level_or_version {
                 pkg.bump(level_or_version, self.metadata.as_deref())?;
             }
+            if index.crate_(&pkg.meta.name).is_some() {
+                // Already published, skip it.  Use `cargo release owner` for one-time updates
+                pkg.ensure_owners = false;
+            }
         }
 
         let (_selected_pkgs, excluded_pkgs) = self.workspace.partition_packages(&ws_meta);