Skip to content

Refactor expect to Result<> mapping#158

Merged
msrd0 merged 4 commits intomainfrom
fix/possible-panic
Feb 19, 2026
Merged

Refactor expect to Result<> mapping#158
msrd0 merged 4 commits intomainfrom
fix/possible-panic

Conversation

@Empty2k12
Copy link
Copy Markdown
Collaborator

Description

Refactors a possible panic to use graceful error handling

Checklist

  • Formatted code using cargo fmt --all
  • Linted code using clippy
    • with reqwest feature: cargo clippy --manifest-path influxdb/Cargo.toml --all-targets --no-default-features --features serde,derive,reqwest-client-rustls -- -D warnings
    • with surf feature: cargo clippy --manifest-path influxdb/Cargo.toml --all-targets --no-default-features --features serde,derive,hyper-client -- -D warnings
  • Updated README.md using cargo doc2readme -p influxdb --expand-macros
  • Reviewed the diff. Did you leave any print statements or unnecessary comments?
  • Any unfinished work that warrants a separate issue captured in an issue with a TODO code comment

@Empty2k12 Empty2k12 force-pushed the main branch 6 times, most recently from 4b5b20b to 239bb3e Compare December 13, 2024 16:50
msrd0
msrd0 previously approved these changes Dec 13, 2024
@Empty2k12 Empty2k12 force-pushed the main branch 6 times, most recently from c241371 to 359fd08 Compare December 13, 2024 17:35
@msrd0 msrd0 merged commit c3dddec into main Feb 19, 2026
35 of 36 checks passed
@msrd0 msrd0 deleted the fix/possible-panic branch February 19, 2026 11:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants