Skip to content

Commit dd564f0

Browse files
committed
refactor: change cleanup
refactor from false to true
1 parent 69458df commit dd564f0

2 files changed

Lines changed: 4 additions & 5 deletions

File tree

src/config.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -120,6 +120,7 @@ pub fn edit_window_title(config: &mut Config, changes_made: &mut bool) {
120120
} else {
121121
config.window_title_support = false;
122122
println!("✅ Window title support disabled.");
123+
*changes_made = true;
123124
return;
124125
}
125126

src/csv.rs

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -114,9 +114,7 @@ fn merge_imported_commands(
114114
config.commands = new_commands;
115115
*changes_made = true;
116116
}
117-
_ => {
118-
*changes_made = false;
119-
}
117+
_ => {}
120118
}
121119
}
122120

@@ -198,13 +196,13 @@ mod tests {
198196
display_name: "ShouldNotAdd".into(),
199197
command: "echo nope".into(),
200198
}];
201-
let mut changed = false;
199+
let mut changed = true;
202200

203201
merge_imported_commands(&mut config, new, "cancel", &mut changed);
204202

205203
assert_eq!(config.commands.len(), 1);
206204
assert_eq!(config.commands[0].display_name, "Keep");
207-
assert!(!changed);
205+
assert!(changed);
208206
}
209207
#[test]
210208
fn test_read_commands_from_csv_valid_csv() {

0 commit comments

Comments
 (0)