From d101baa0ef35d0836f7d15b3a5562f430ba2c74f Mon Sep 17 00:00:00 2001 From: Ed Page Date: Fri, 31 Jan 2025 10:59:49 -0600 Subject: [PATCH] fix(commit): Use correct default commit message with implicit consolidated commits Found when investigating #701 --- src/steps/release.rs | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/steps/release.rs b/src/steps/release.rs index 1341153d5..d1a8a73bf 100644 --- a/src/steps/release.rs +++ b/src/steps/release.rs @@ -59,7 +59,7 @@ impl ReleaseStep { // When evaluating dependency ordering, we need to consider optional dependencies .features(cargo_metadata::CargoOpt::AllFeatures) .exec()?; - let ws_config = config::load_workspace_config(&self.config, &ws_meta)?; + let mut ws_config = config::load_workspace_config(&self.config, &ws_meta)?; let mut pkgs = plan::load(&self.config, &ws_meta)?; for pkg in pkgs.values_mut() { @@ -193,6 +193,7 @@ impl ReleaseStep { let mut failed = false; let consolidate_commits = super::consolidate_commits(&selected_pkgs, &excluded_pkgs)?; + ws_config.consolidate_commits = Some(consolidate_commits); // STEP 0: Help the user make the right decisions. failed |= !super::verify_git_is_clean(