Skip to content

std::result::Result#31

Open
afeistel wants to merge 1 commit into
softprops:masterfrom
parrrate:master
Open

std::result::Result#31
afeistel wants to merge 1 commit into
softprops:masterfrom
parrrate:master

Conversation

@afeistel

@afeistel afeistel commented Oct 1, 2025

Copy link
Copy Markdown

What did you implement:

Replaces Result with std::result::Result in the derived code.

Why: had errors because of type Result<T> = std::result::Result<T, Error> (a common thing to do).

We really should also prefix everything with :: at some point (::std::something instead of std::something). For now, kept it consistent with existing code.

How did you verify your change:

Ran tests (they passed), ran the example (it didn't error out), built the code where I needed that change made (it compiled correctly).

What (if anything) would need to be called out in the CHANGELOG for the next release:

### Fixed

- Having a `Result` type alias no longer interferes with `#[derive(Recap)]`

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