Skip to content

Improve error reporting for illegal variable names#762

Merged
mmarx merged 1 commit intomainfrom
better-fstring-errors
Feb 18, 2026
Merged

Improve error reporting for illegal variable names#762
mmarx merged 1 commit intomainfrom
better-fstring-errors

Conversation

@mmarx
Copy link
Member

@mmarx mmarx commented Feb 18, 2026

Improve the reporting of illegal variable names, e.g., in fstrings (previously, this would report “expected )” at the opening "):

[01] Error: expected `letter or digit`
   ╭─[ /dev/shm/foo.rls:2:15 ]
   │
 2 │ foo(f"test: {?_test}") :- bar(_), ?_test = 42.
   │               │
   │               ╰─ expected `letter or digit`
───╯

@mmarx mmarx added this to the Release 0.10.0 milestone Feb 18, 2026
@mmarx mmarx requested a review from aannleax February 18, 2026 14:23
@mmarx mmarx self-assigned this Feb 18, 2026
@mmarx mmarx added the Parser Rule-Parser-Related Issue label Feb 18, 2026
@mmarx mmarx added this to nemo Feb 18, 2026
@github-project-automation github-project-automation bot moved this to Todo in nemo Feb 18, 2026
Copy link
Member

@aannleax aannleax left a comment

Choose a reason for hiding this comment

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

Looks good :)

@github-project-automation github-project-automation bot moved this from Todo to In Progress in nemo Feb 18, 2026
@mmarx mmarx merged commit d6341b7 into main Feb 18, 2026
8 checks passed
@mmarx mmarx deleted the better-fstring-errors branch February 18, 2026 14:39
@github-project-automation github-project-automation bot moved this from In Progress to Done in nemo Feb 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Parser Rule-Parser-Related Issue

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants