Skip to content

Commit e7ad070

Browse files
romtsnclaude
andcommitted
fix(code-mappings): Include batch errors in summary error count
The Errors count in the summary line now includes both per-mapping errors and batch-level transport failures, matching the bail message. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 39f6c27 commit e7ad070

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

src/commands/code_mappings/upload.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -114,13 +114,13 @@ pub fn execute(matches: &ArgMatches) -> Result<()> {
114114
println!("{err}");
115115
}
116116

117+
let total_errors = merged.errors + merged.batch_errors.len() as u64;
117118
println!(
118119
"Created: {}, Updated: {}, Errors: {}",
119-
merged.created, merged.updated, merged.errors
120+
merged.created, merged.updated, total_errors
120121
);
121122

122-
if merged.errors > 0 || !merged.batch_errors.is_empty() {
123-
let total_errors = merged.errors + merged.batch_errors.len() as u64;
123+
if total_errors > 0 {
124124
bail!("{total_errors} error(s) during upload. See details above.");
125125
}
126126

0 commit comments

Comments
 (0)