Skip to content

[pull] master from tamasfe:master#391

Merged
pull[bot] merged 1 commit intoturkdevops:masterfrom
tamasfe:master
Oct 22, 2025
Merged

[pull] master from tamasfe:master#391
pull[bot] merged 1 commit intoturkdevops:masterfrom
tamasfe:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull Bot commented Oct 22, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

According to the toml standard, bare keys may only contain the
characters A-Za-z0-9_- and they must contain at least one character.
This allows identifiers to look like dates, e.g. 2024-01-01.

Due to a bug in the lexing library logos, some-like identifiers are
incorrectly being parsed as INTEGER tokens, while date-like identifiers
in table names are being parsed as DATE tokens as they should be. Due to
this bug, date-like identifiers were being accepted in some situations
by taplo while they were rejected in other situations.

This commit only allows DATE tokens in identifiers (as they should be,
according to the standard). It does not fix the underlying bug that
sometimes produces incorrect INTEGER tokens instead of DATE tokens.
@pull pull Bot locked and limited conversation to collaborators Oct 22, 2025
@pull pull Bot added the ⤵️ pull label Oct 22, 2025
@pull pull Bot merged commit d4d1fcb into turkdevops:master Oct 22, 2025
1 check failed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant