From 52f0667777e0c4ed1b343a0f216e048bd98ebb56 Mon Sep 17 00:00:00 2001 From: Lucien Cartier-Tilet Date: Tue, 31 Mar 2026 14:05:52 +0200 Subject: [PATCH] refactor(BreakingChange): rename method ignore to is_absent Method `ignore` did not carry its meaning well by the way it is named. This commit renames it to `is_absent` to clearly state this method returns whether we have a breaking change. --- src/commit/types/breaking_change.rs | 2 +- src/commit/types/message.rs | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/commit/types/breaking_change.rs b/src/commit/types/breaking_change.rs index 48d0abc..c28b170 100644 --- a/src/commit/types/breaking_change.rs +++ b/src/commit/types/breaking_change.rs @@ -31,7 +31,7 @@ pub enum BreakingChange { } impl BreakingChange { - pub fn ignore(&self) -> bool { + pub fn is_absent(&self) -> bool { matches!(self, BreakingChange::No) } diff --git a/src/commit/types/message.rs b/src/commit/types/message.rs index b9dc058..f74e8cc 100644 --- a/src/commit/types/message.rs +++ b/src/commit/types/message.rs @@ -76,7 +76,7 @@ impl ConventionalCommit { pub fn first_line_len(&self) -> usize { self.commit_type.len() + self.scope.header_segment_len() - + if self.breaking_change.ignore() { 0 } else { 1 } + + if self.breaking_change.is_absent() { 0 } else { 1 } + 2 // ": " + self.description.len() }