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

Revert #50757 - Object type literal with string literal property in contextual typing position causes language service error on all literal type references #51159

Closed
wants to merge 0 commits into from

Conversation

a-tarasyuk
Copy link
Contributor

@a-tarasyuk a-tarasyuk commented Oct 13, 2022

@typescript-bot typescript-bot added the For Uncommitted Bug PR for untriaged, rejected, closed or missing bug label Oct 13, 2022
@typescript-bot
Copy link
Collaborator

This PR doesn't have any linked issues. Please open an issue that references this PR. From there we can discuss and prioritise.

@a-tarasyuk
Copy link
Contributor Author

/cc @sandersn

@DanielRosenwasser
Copy link
Member

DanielRosenwasser commented Oct 13, 2022

I think the current fix is fine. I'm going to revert and come up with a broader fix with @sandersn in #51066.

@sandersn sandersn added this to Not started in PR Backlog Oct 27, 2022
@sandersn
Copy link
Member

@DanielRosenwasser As I understand your comment, you want to keep the code as-is until #51066 (or something similar) is done, right? I guess that #51066 will revert #50757 when it's ready.

If that's the case, let's close this PR.

@sandersn sandersn moved this from Not started to Waiting on reviewers in PR Backlog Oct 27, 2022
@DanielRosenwasser
Copy link
Member

Yup, sounds good.

@a-tarasyuk a-tarasyuk closed this Oct 28, 2022
PR Backlog automation moved this from Waiting on reviewers to Done Oct 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
For Uncommitted Bug PR for untriaged, rejected, closed or missing bug
Projects
PR Backlog
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants