Skip to content

fix: ban all unwrap to improve error handling#71

Merged
reneca merged 1 commit intomainfrom
ban_unwrap
Apr 2, 2026
Merged

fix: ban all unwrap to improve error handling#71
reneca merged 1 commit intomainfrom
ban_unwrap

Conversation

@reneca
Copy link
Copy Markdown
Contributor

@reneca reneca commented Mar 30, 2026

Now all Cargo.toml contain:

[lints.clippy]
unwrap_used = "deny"

To forbid the use of unwrap in the code.

  • fix some crash that could occur
  • improve error handling
  • reduce crash during runtime

Signed-off-by: Jeremy HERGAULT <jeremy.hergault@worldline.com>
@reneca reneca requested review from Timmy80 and oschijns March 30, 2026 14:34
@reneca reneca self-assigned this Mar 30, 2026
@reneca reneca added bug Something isn't working enhancement New feature or request labels Mar 30, 2026
@reneca reneca merged commit 76b1887 into main Apr 2, 2026
7 checks passed
@reneca reneca deleted the ban_unwrap branch April 2, 2026 11:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants