Replace process::exit(1) with proper error propagation #17

New Issue

GKaszewski · 2026-07-25T11:18:07Z

GKaszewski commented

2026-07-25 11:18:07 +00:00

Severity: Low

crates/presentation/src/lib.rs lines 59 and 245: std::process::exit(1) is called directly, which skips destructors and Drop implementations.

For the check mode (line 59) and run_diff (line 245), return an Err instead — the function return types are already Result<()>.

**Severity: Low** `crates/presentation/src/lib.rs` lines 59 and 245: `std::process::exit(1)` is called directly, which skips destructors and `Drop` implementations. For the check mode (line 59) and `run_diff` (line 245), return an `Err` instead — the function return types are already `Result<()>`.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: GKaszewski/archlens#17