Skip to content

Conversation

@mellowagain
Copy link
Contributor

@mellowagain mellowagain commented Jun 29, 2023

previously, we only checked if the outer type was an impl Trait. now we also check the inner type, if such one exists

fixes #121

previously, we only checked if the other type was an `impl Trait`. now we also check the inner type, if such one exists
@mellowagain mellowagain requested review from a team and gagbo June 29, 2023 13:41
Copy link
Contributor

@gagbo gagbo left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for the test

@mellowagain
Copy link
Contributor Author

im actually running into compile errors with a local project using this very autometrics branch, so not yet merging this until i figured out that bug

Copy link
Contributor

@emschwartz emschwartz left a comment

Choose a reason for hiding this comment

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

Looks good. Can you add a changelog entry?

@mellowagain
Copy link
Contributor Author

im actually running into compile errors with a local project using this very autometrics branch, so not yet merging this until i figured out that bug

the bug is now fixed. turns out i didnt correctly insert :: between path segments. i added tests to catch such a thing in the future

@mellowagain mellowagain enabled auto-merge (squash) July 3, 2023 09:24
@mellowagain mellowagain merged commit cb7b6b8 into main Jul 3, 2023
@mellowagain mellowagain deleted the issue-121 branch July 3, 2023 09:26
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.

returning Result<impl Trait> from a #[autometrics]'d function results in compile error

4 participants