Skip to content

Show type info of yield #32283

Open
Open

Description

@falsandtru

Ron Buckton (@rbuckton) yield keyword should show its argument type and return type when a cursor is put on there.

Activity

  1. sandersn commented on Jul 9, 2019

    @sandersn
    Member

    Good idea, although I don't think we produce yield's type when checking, although this might have recently changed.

  2. falsandtru commented on Jul 9, 2019

    @falsandtru
    ContributorAuthor

    In fact, finding a break of type checking such as #32248 (yield wrongly returns any type.) is impossible when we can't directly check a type of yield. When yield argument type is any, we can't know it. When yield return type is any, we can't know it without seeing an inferred type via a variable.

  3. falsandtru commented on Jul 19, 2019

    @falsandtru
    ContributorAuthor

    Nathan Shively-Sanders (@sandersn) Ron Buckton (@rbuckton) I recommend you to release this feature with the next release to make complete user experience of enhanced generators.

  4. added
    Effort: ModerateRequires experience with the TypeScript codebase, but feasible. Harder than "Effort: Casual".
    Needs ProposalThis issue needs a plan that clarifies the finer details of how it could be implemented.
    and removed on Jul 23, 2019
  5. Kingwl commented on Jul 30, 2019

    @Kingwl
    Contributor

    What is the expected display?
    Similar to NewKeyword? or something like (TNext) yield (T)

  6. falsandtru commented on Jul 30, 2019

    @falsandtru
    ContributorAuthor

    I expect it.

  7. Kingwl commented on Aug 12, 2019

    @Kingwl
    Contributor

    Expect which one🤦🏻‍♂️

  8. falsandtru commented on Aug 12, 2019

    @falsandtru
    ContributorAuthor

    Ah, I meant (TNext) yield (T).

  9. Kingwl commented on Aug 12, 2019

    @Kingwl
    Contributor

    Daniel Rosenwasser (@DanielRosenwasser) Any suggestions?🙋🏻‍♂️

  10. DanielRosenwasser commented on Mar 22, 2024

    @DanielRosenwasser
    Member

    Similar to #11261 which is about quick info on return.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Domain: LS: Quick Infoe.g. hover text, tool-tips, and tooltips.Effort: ModerateRequires experience with the TypeScript codebase, but feasible. Harder than "Effort: Casual".Experience EnhancementNoncontroversial enhancementsHelp WantedYou can do thisNeeds ProposalThis issue needs a plan that clarifies the finer details of how it could be implemented.

    Type

    No type

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions