Skip to content

Allow unescaped paragraph and line separators in string literal#53869

Merged
jakebailey merged 4 commits intomicrosoft:mainfrom
liviu-gheorghe:allowUnescapedParagraphAndLineSeparatorsInStringLiteral
Sep 15, 2023
Merged

Allow unescaped paragraph and line separators in string literal#53869
jakebailey merged 4 commits intomicrosoft:mainfrom
liviu-gheorghe:allowUnescapedParagraphAndLineSeparatorsInStringLiteral

Conversation

@liviu-gheorghe
Copy link
Copy Markdown
Contributor

Fixes #53519

@typescript-bot typescript-bot added the For Uncommitted Bug PR for untriaged, rejected, closed or missing bug label Apr 17, 2023
@typescript-bot
Copy link
Copy Markdown
Collaborator

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

@liviu-gheorghe
Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree

@jakebailey
Copy link
Copy Markdown
Member

I'm trying to get a grasp of this and here's what esbuild does.

So, I think this is equivalent?

Copy link
Copy Markdown
Member

@jakebailey jakebailey left a comment

Choose a reason for hiding this comment

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

Should tests/cases/compiler/sourceMap-LineBreaks.ts have changed? My impression is that it includes these characters, but, nothing happens?

…ph and line separators in string literal
@jakebailey jakebailey merged commit e654f96 into microsoft:main Sep 15, 2023
snovader pushed a commit to EG-A-S/TypeScript that referenced this pull request Sep 23, 2023
…osoft#53869)

Co-authored-by: Liviu Gheorghe <liviu.gheorghe@ivfuture.uk>
Co-authored-by: Jake Bailey <5341706+jakebailey@users.noreply.github.com>
@microsoft microsoft locked as resolved and limited conversation to collaborators Oct 22, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

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

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

TypeScript incorrectly handles naughty strings in json

5 participants