Skip to content

Fix issue #6478 (bug in the language services) #6507

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

Merged
merged 2 commits into from
Jan 16, 2016

Conversation

jkillian
Copy link

I made the changes suggested in #6478 (check that symbol is not undefined right away) and found a fairly minimal repro which I added as a fourslash test case.

Edit: The paragraph below is out of date now :)
The last commit, which reverts package.json to use the latest nightly instead of a fixed version, causes this build to break since the latest nightly still has bugs in it. Most likely that revert should be done tomorrow, after this PR is in and a new, working nightly is released. I'll remove the revert from here so that can be done later if requested

@DanielRosenwasser
Copy link
Member

Hey @jkillian can you hard reset your branch to 047c62c and force push? We can perform the revert tomorrow after the nightly is updated.

@jkillian
Copy link
Author

Yep, done!

DanielRosenwasser added a commit that referenced this pull request Jan 16, 2016
Fix issue #6478 (bug in the language services)
@DanielRosenwasser DanielRosenwasser merged commit 4673b57 into microsoft:master Jan 16, 2016
@jkillian jkillian deleted the fix-prop-bug branch January 16, 2016 04:39
@microsoft microsoft locked and limited conversation to collaborators Jun 19, 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.

3 participants