Skip to content

toml, tests: support v2.0.0 of the test-suite at toml-lang/toml-test@229ce2e - #26101

Merged
spytheman merged 1 commit into
vlang:masterfrom
larpon:toml/ffwd-to-229ce2e
Dec 24, 2025
Merged

toml, tests: support v2.0.0 of the test-suite at toml-lang/toml-test@229ce2e#26101
spytheman merged 1 commit into
vlang:masterfrom
larpon:toml/ffwd-to-229ce2e

Conversation

@larpon

@larpon larpon commented Dec 24, 2025

Copy link
Copy Markdown
Contributor

This PR "fast forwards" to handle toml-lang/toml-test@229ce2e which is the latest commit as of today.

A version 2.0 of the suite was released last week so hopefully the setup should hold up for a while.

This comes at the small "expense" of having to add a few new entries to the invalid_exceptions list (I need more time/motivation before fixing those) where a few entries was also renamed to use 0 (zero) prefixes for numbers in the file names.

I've also included a small fix for detecting invalid parsed time, because... well... it's Christmas time and I felt joy doing it 🙂🎄🎅🏻 (fixes concrete test: vlib/toml/tests/testdata/toml_lang/tests/invalid/datetime/second-trailing-dot.toml)

Merry Christmas to everyone on the V team, that celebrates Christmas.

@spytheman spytheman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you @larpon .
Excellent work.

Merry Christmas to you and your family.

@spytheman
spytheman merged commit fe6162e into vlang:master Dec 24, 2025
72 checks passed
@larpon
larpon deleted the toml/ffwd-to-229ce2e branch December 24, 2025 12:14
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.

2 participants