Skip to content

docs: add # Errors sections for clippy pedantic compliance#19

Merged
refcell merged 1 commit intomainfrom
reviewer-alpha/errors-doc-final
Feb 1, 2026
Merged

docs: add # Errors sections for clippy pedantic compliance#19
refcell merged 1 commit intomainfrom
reviewer-alpha/errors-doc-final

Conversation

@refcell
Copy link
Owner

@refcell refcell commented Feb 1, 2026

Summary

  • Add # Errors documentation to Decode and Encode trait methods in codec.rs
  • Add # Errors documentation to Handle::join() in runtime.rs

These additions satisfy clippy's missing_errors_doc pedantic lint for public functions returning Result.

- Add backticks to ErrorPayload in error.rs doc (clippy doc_markdown)
- Inline format args in to_error_payload() (clippy uninlined_format_args)
- Add # Errors documentation to Decode and Encode trait methods (clippy missing_errors_doc)
- Add # Errors documentation to Handle::join() (clippy missing_errors_doc)
@refcell refcell merged commit 1e82622 into main Feb 1, 2026
17 checks passed
@refcell refcell deleted the reviewer-alpha/errors-doc-final branch February 1, 2026 23:41
@refcell
Copy link
Owner Author

refcell commented Feb 1, 2026

Review Note (reviewer-gamma):

This PR's changes are identical to part of PR #18 (TokioContext tests). Both PRs modify:

  • crates/traits/src/codec.rs - same # Errors sections
  • crates/traits/src/runtime.rs - same # Errors section

Recommendation: If #18 is merged first, this PR becomes redundant and should be closed. If you prefer to keep the changes separate, one PR will need to rebase after the other merges.

The doc changes themselves look correct! ✅

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.

1 participant