Skip to content

Conversation

@jessetrinity
Copy link
Contributor

Reverting #43460 and #40884 to cherry pick to 4.3 until we verify that we have a good fix in #44158.

@typescript-bot typescript-bot added Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug labels May 19, 2021
@jessetrinity jessetrinity force-pushed the revertBraceMatchingDiagnostics branch from ce216c3 to f537aef Compare May 19, 2021 23:03
@jessetrinity jessetrinity merged commit 5770434 into microsoft:master May 19, 2021
@jessetrinity
Copy link
Contributor Author

@typescript-bot cherry-pick this to release-4.3

1 similar comment
@jessetrinity
Copy link
Contributor Author

@typescript-bot cherry-pick this to release-4.3

@jessetrinity jessetrinity deleted the revertBraceMatchingDiagnostics branch May 20, 2021 00:34
jessetrinity pushed a commit to jessetrinity/TypeScript that referenced this pull request May 20, 2021
* Revert "Only issue matching token errors on non-dupe locations (microsoft#43460)"

This reverts commit 76a2ae3.

* Revert "Adding Diagnostic message for missing ']' and ')' in Array literal and conditional statements (microsoft#40884)"

This reverts commit 555ef73.

* re-add clobbered merge lines
DanielRosenwasser pushed a commit that referenced this pull request May 20, 2021
* Revert "Only issue matching token errors on non-dupe locations (#43460)"

This reverts commit 76a2ae3.

* Revert "Adding Diagnostic message for missing ']' and ')' in Array literal and conditional statements (#40884)"

This reverts commit 555ef73.

* re-add clobbered merge lines
jessetrinity added a commit to jessetrinity/TypeScript that referenced this pull request May 27, 2021
sandersn added a commit that referenced this pull request Mar 16, 2022
…brackets (#44158)

* Revert "Revert #43460 and #40884 (#44175)"

This reverts commit 5770434.

* fix missing opening brace match error

* refactor parseExpectedMatchingBrackets

* use getNodePos

* accept baselines

* delete mistakenly added files

* Revert getNodePos addition

Co-authored-by: Nathan Shively-Sanders <[email protected]>
@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

Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants