Skip to content

feat(sdk): mark fallible calls as must use - #86

Merged
GianIac merged 1 commit into
GianIac:mainfrom
lntutor:feat/sdk-must-use-results
Jul 30, 2026
Merged

feat(sdk): mark fallible calls as must use#86
GianIac merged 1 commit into
GianIac:mainfrom
lntutor:feat/sdk-must-use-results

Conversation

@lntutor

@lntutor lntutor commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add an explicit #[must_use] diagnostic to all 38 public fallible SDK calls
  • use one consistent message so ignored results explain how callers should respond
  • leave infallible SDK functions and runtime behavior unchanged

Fixes #85

Verification

  • cargo fmt --all -- --check
  • cargo clippy --workspace --all-targets -- -D warnings
  • cargo test --workspace

Signed-off-by: Loi Nguyen <vanloi999@gmail.com>
@GianIac

GianIac commented Jul 30, 2026

Copy link
Copy Markdown
Owner

Thanks! If you'd like to contribute to other things as well, you're more than welcome!

@GianIac
GianIac merged commit b529293 into GianIac:main Jul 30, 2026
16 checks passed
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.

sdk: add #[must_use] to fallible SDK functions returning Result

2 participants