Skip to content

Conversation

@0101
Copy link
Contributor

@0101 0101 commented Apr 20, 2023

Redundant return type hints are shown when annotation is present with a generic or type parameters.

@0101 0101 marked this pull request as ready for review April 20, 2023 11:57
@0101 0101 requested a review from a team as a code owner April 20, 2023 11:57
Copy link
Contributor

@psfinaki psfinaki left a comment

Choose a reason for hiding this comment

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

So that's a bug in the compiler actually? I am surprised nothing else is affected but well okay :)

@0101
Copy link
Contributor Author

0101 commented Apr 20, 2023

So that's a bug in the compiler actually? I am surprised nothing else is affected but well okay :)

No, I made that function for return type hints (actually you made me put it there 😀) and nothing else uses it. I copied that particular part from IsTypeAnnotationGivenAtPosition which was already there before. But it didn't cover these cases.

@0101 0101 merged commit 58cc1a4 into dotnet:main Apr 20, 2023
@psfinaki
Copy link
Contributor

Ah right. Okay then, thanks!

vzarytovskii added a commit that referenced this pull request Apr 25, 2023
Co-authored-by: vzarytovskii <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Kevin Ransom (msft) <[email protected]>
Co-authored-by: Vlad Zarytovskii <[email protected]>
Co-authored-by: Tomas Grosup <[email protected]>
Co-authored-by: Florian Verdonck <[email protected]>
Co-authored-by: Petr <[email protected]>
Co-authored-by: Petr Pokorny <[email protected]>
Fixing hints for custom ops (#15119)
Fix bug in return type hints (#15124)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants