Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

fix(33836): VSCode - generate accessors fixup on readonly field removes modifier#36543

Merged
jessetrinity merged 2 commits into
microsoft:mastermicrosoft/TypeScript:masterfrom
a-tarasyuk:bug/33836a-tarasyuk/TypeScript:bug/33836Copy head branch name to clipboard
May 13, 2020
Merged

fix(33836): VSCode - generate accessors fixup on readonly field removes modifier#36543
jessetrinity merged 2 commits into
microsoft:mastermicrosoft/TypeScript:masterfrom
a-tarasyuk:bug/33836a-tarasyuk/TypeScript:bug/33836Copy head branch name to clipboard

Conversation

@a-tarasyuk

Copy link
Copy Markdown
Contributor

Fixes #33836

@jessetrinity jessetrinity self-requested a review May 13, 2020 18:49

@jessetrinity jessetrinity left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me.

@jessetrinity jessetrinity merged commit 7171125 into microsoft:master May 13, 2020
cangSDARM added a commit to cangSDARM/TypeScript that referenced this pull request May 14, 2020
* upstream/master:
  Perform intersection reduction before and after getApparentType (microsoft#38565)
  fix(33836): allow readonly modifier for a field with only get accessor (microsoft#36543)
  Fix build type error (microsoft#38525)
@microsoft microsoft locked as resolved and limited conversation to collaborators Oct 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

For Backlog Bug PRs that fix a backlog bug

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

VSCode - generate accessors fixup on readonly field removes modifier

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.