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

Lanugage Service support for union types #861

Merged
merged 12 commits into from
Oct 11, 2014
Merged

Lanugage Service support for union types #861

merged 12 commits into from
Oct 11, 2014

Conversation

mhegazy
Copy link
Contributor

@mhegazy mhegazy commented Oct 9, 2014

Completions kind, Quick Info, go to def, and find all refss

@danquirk
Copy link
Member

danquirk commented Oct 9, 2014

Probably could use some fourslash tests for generic type inference and type guards

…ies of the union type but are properties on of one of its constituant types
@DanielRosenwasser
Copy link
Member

👍

var declarations = symbol.getDeclarations();

// Handel union properties
if (symbol.flags & SymbolFlags.UnionProperty) {
Copy link
Contributor

Choose a reason for hiding this comment

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

handle

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

mhegazy added a commit that referenced this pull request Oct 11, 2014
Lanugage Service support for union types
@mhegazy mhegazy merged commit eee1602 into unionTypes Oct 11, 2014
@mhegazy mhegazy deleted the unionTypesLS branch October 14, 2014 00:15
@microsoft microsoft locked and limited conversation to collaborators Jun 18, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants