Skip to content

Conversation

timreichen
Copy link
Contributor

Seems like all tests still pass without \b.

@timreichen timreichen requested a review from kt3k as a code owner May 25, 2025 13:26
@github-actions github-actions bot added the toml label May 25, 2025
Copy link

codecov bot commented May 25, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.70%. Comparing base (ab916fd) to head (6eaea50).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #6682   +/-   ##
=======================================
  Coverage   94.69%   94.70%           
=======================================
  Files         562      563    +1     
  Lines       46651    46676   +25     
  Branches     6570     6573    +3     
=======================================
+ Hits        44177    44203   +26     
  Misses       2431     2431           
+ Partials       43       42    -1     

☔ 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.

@kt3k
Copy link
Member

kt3k commented May 26, 2025

After this change parse("1 = truea = 1") parse to { "1": true, a: 1 } while it raises syntax error correctly on main. I think we need more test to check this

@timreichen
Copy link
Contributor Author

After this change parse("1 = truea = 1") parse to { "1": true, a: 1 } while it raises syntax error correctly on main. I think we need more test to check this

#6663 (comment)

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.

2 participants