Skip to content

fix: handle whitespace correctly#3110

Merged
peter-jerry-ye merged 1 commit intomainfrom
zihang/fix-json
Jan 5, 2026
Merged

fix: handle whitespace correctly#3110
peter-jerry-ye merged 1 commit intomainfrom
zihang/fix-json

Conversation

@peter-jerry-ye
Copy link
Collaborator

Historically, the json parser was ported from moonbitlang/x/json5. As a result, some of the code was actually JSON 5 compatible instead of JSON compatible. While developing #3109 this inconsistency was exposed.

This PR fixes the issue and update the test case.

@coveralls
Copy link
Collaborator

Pull Request Test Coverage Report for Build 2262

Details

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.002%) to 98.202%

Totals Coverage Status
Change from base Build 2259: -0.002%
Covered Lines: 10597
Relevant Lines: 10791

💛 - Coveralls

@peter-jerry-ye peter-jerry-ye merged commit b6100dd into main Jan 5, 2026
18 checks passed
@peter-jerry-ye peter-jerry-ye deleted the zihang/fix-json branch January 5, 2026 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants