Skip to content

Commit d52a788

Browse files
committed
Address TODO comment
1 parent 091b090 commit d52a788

File tree

2 files changed

+2
-10
lines changed

2 files changed

+2
-10
lines changed

jruby_executable/src/bin/jruby_build.rs

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -141,11 +141,7 @@ fn jruby_build(args: &Args) -> Result<(), Box<dyn Error>> {
141141
atomic_inventory_update(&inventory, |inventory| {
142142
for prior in &inventory.artifacts {
143143
if let Err(error) = artifact_same_url_different_checksum(prior, &artifact) {
144-
// TODO: Investigate bullet stream ownership
145-
println!(
146-
"{}",
147-
style::important(format!("!!!!!!!!!! Error updating inventory: {error}"))
148-
);
144+
print::error(format!("Error updating inventory\n\nError: {error}"));
149145

150146
fs_err::remove_file(&sha_seven_path)?;
151147
return Err(error);

ruby_executable/src/bin/ruby_build.rs

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -165,11 +165,7 @@ fn ruby_build(args: &RubyArgs) -> Result<(), Box<dyn std::error::Error>> {
165165
atomic_inventory_update(&inventory, |inventory| {
166166
for prior in &inventory.artifacts {
167167
if let Err(error) = artifact_same_url_different_checksum(prior, &artifact) {
168-
// TODO: Investigate bullet stream ownership
169-
println!(
170-
"{}",
171-
style::important(format!("!!!!!!!!!! Error updating inventory: {error}"))
172-
);
168+
print::error(format!("Error updating inventory\n\nError: {error}"));
173169

174170
fs_err::remove_file(&sha_seven_path)?;
175171
return Err(error);

0 commit comments

Comments
 (0)