Skip to content

Fix outdated Try_ConvertResult documentation#200

Open
wenxiaojie1 wants to merge 1 commit into
MusicalNinjaDad:mainfrom
wenxiaojie1:docs/replace-try-convert-result
Open

Fix outdated Try_ConvertResult documentation#200
wenxiaojie1 wants to merge 1 commit into
MusicalNinjaDad:mainfrom
wenxiaojie1:docs/replace-try-convert-result

Conversation

@wenxiaojie1

Copy link
Copy Markdown
Contributor

Fixes #175.\n\nReplaces the removed Try_ConvertResult derive in the published README and crate-level API example with the current #[FromResidual(Result<_, Self::Residual>)] pattern.\n\nValidation:\n- git diff --check\n- Confirmed no remaining Try_ConvertResult references in README.md or crates/try_v2/src/lib.rs\n\ncargo doc was not run locally because this Windows environment has no Rust toolchain; the repository CI remains the compilation check.

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

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.

Docs reference removed Try_ConvertResult instead of new FromResidual approach

1 participant