Skip to content

Conversation

@l0ngvh
Copy link
Contributor

@l0ngvh l0ngvh commented May 18, 2025

Summary

Allows omitting value in an explicit mapping entry

Test Plan

The parser can now parse the following example:

? a
? b

Which is equivalent to

{
  "a": null,
  "b": null
}

@github-actions github-actions bot added the A-Parser Area: parser label May 18, 2025
@l0ngvh l0ngvh marked this pull request as ready for review May 18, 2025 07:51
@l0ngvh l0ngvh requested review from a team May 18, 2025 07:51
@codspeed-hq
Copy link

codspeed-hq bot commented May 18, 2025

CodSpeed Performance Report

Merging #6054 will not alter performance

Comparing vohoanglong0107:yaml-explicit-mapping-empty-value (d88a01e) with main (a82a1f2)

Summary

✅ 94 untouched benchmarks
🆕 1 new benchmarks

Benchmarks breakdown

Benchmark BASE HEAD Change
🆕 html_formatter[index_1033418810622582172.html] N/A 462.7 µs N/A

@l0ngvh l0ngvh merged commit 77eb494 into biomejs:main May 18, 2025
13 checks passed
@l0ngvh l0ngvh deleted the yaml-explicit-mapping-empty-value branch May 18, 2025 11:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Parser Area: parser

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants