Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ignore return-type annotations #35

Merged
merged 2 commits into from Apr 26, 2021
Merged

ignore return-type annotations #35

merged 2 commits into from Apr 26, 2021

Conversation

laanak08
Copy link
Contributor

This fixes an issue where any function with return-type annotations will get processed through parse_obj_as, and inevitably throw a validation exception, which will be returned as a very confusing error message where the "loc" property is "return".

This fixes an issue where any function with return-type annotations will get processed through `parse_obj_as`, and inevitably throw a validation exception, which will be returned as a very confusing error message where the "loc" property is "return".
@laanak08 laanak08 marked this pull request as draft April 25, 2021 17:45
@bauerji
Copy link
Owner

bauerji commented Apr 26, 2021

Thank you for contribution. Could you please add tests to describe this behaviour?

@sonarcloud
Copy link

sonarcloud bot commented Apr 26, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@laanak08 laanak08 marked this pull request as ready for review April 26, 2021 13:06
@bauerji bauerji merged commit 6537130 into bauerji:master Apr 26, 2021
@bauerji
Copy link
Owner

bauerji commented Apr 26, 2021

Released in version 0.7.2

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.

None yet

2 participants