Skip to content

fix(json): correctly escape unicode characters#9419

Merged
h-a-n-a merged 1 commit intoweb-infra-dev:mainfrom
inottn:fix/json-escape
Feb 21, 2025
Merged

fix(json): correctly escape unicode characters#9419
h-a-n-a merged 1 commit intoweb-infra-dev:mainfrom
inottn:fix/json-escape

Conversation

@inottn
Copy link
Collaborator

@inottn inottn commented Feb 21, 2025

Summary

fix #9415

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@github-actions github-actions bot added the release: bug fix release: bug related release(mr only) label Feb 21, 2025
@netlify
Copy link

netlify bot commented Feb 21, 2025

Deploy Preview for rspack canceled.

Built without sensitive environment variables

Name Link
🔨 Latest commit 68ee663
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/67b7d8f95d87a5000893324b

@codspeed-hq
Copy link

codspeed-hq bot commented Feb 21, 2025

CodSpeed Performance Report

Merging #9419 will not alter performance

Comparing inottn:fix/json-escape (68ee663) with main (18420bb)

Summary

✅ 7 untouched benchmarks

@chenjiahan chenjiahan requested a review from h-a-n-a February 21, 2025 04:03
Copy link
Contributor

@h-a-n-a h-a-n-a left a comment

Choose a reason for hiding this comment

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

Thanks

@h-a-n-a h-a-n-a merged commit bc66946 into web-infra-dev:main Feb 21, 2025
31 checks passed
@inottn inottn deleted the fix/json-escape branch February 21, 2025 05:42
@dgaponov
Copy link
Contributor

Thank you

@h-a-n-a h-a-n-a mentioned this pull request Feb 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release: bug fix release: bug related release(mr only)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Incorrect escape Unicode chars in json

3 participants