Skip to content

fix(yaml): quote leading-zero numeric strings#7144

Closed
fibibot wants to merge 1 commit into
mainfrom
orch/issue-73
Closed

fix(yaml): quote leading-zero numeric strings#7144
fibibot wants to merge 1 commit into
mainfrom
orch/issue-73

Conversation

@fibibot
Copy link
Copy Markdown

@fibibot fibibot commented May 16, 2026

Summary

  • Quote leading-zero numeric-looking strings in YAML stringify output so values like 08 and 09 are not emitted as plain scalars.
  • Add regression coverage for 00, 07, 08, 09, 010, and -08.

Closes bartlomieju/orchid-inbox#73

Testing

  • deno fmt --check
  • deno lint (passed with existing workspace-member warning for @std/crypto)
  • deno test yaml/stringify_test.ts
  • deno test -A --trace-leaks --coverage --doc --clean yaml

deno task test was attempted, but the VM killed the process with exit 137 during the parallel full suite before any failure was reported. A full non-parallel equivalent was also attempted and stopped at the VM level before reporting a test failure.

@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.61%. Comparing base (5ea9159) to head (72d1754).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7144   +/-   ##
=======================================
  Coverage   94.61%   94.61%           
=======================================
  Files         634      634           
  Lines       51830    51847   +17     
  Branches     9341     9347    +6     
=======================================
+ Hits        49037    49054   +17     
  Misses       2218     2218           
  Partials      575      575           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants