Skip to content

change "from eval(...)" to "from f'{...}'#6450

Merged
mccanne merged 2 commits intomainfrom
from-fstring
Dec 16, 2025
Merged

change "from eval(...)" to "from f'{...}'#6450
mccanne merged 2 commits intomainfrom
from-fstring

Conversation

@mccanne
Copy link
Collaborator

@mccanne mccanne commented Dec 16, 2025

This commit changes the "robot from" syntax from the eval() structure to a simple f-string. This recognizes that entities in from operators are always identified as strings. The ability to scan the entities in an array of strings is no longer available with this change, but this can always be emulated by doing an "over" upstream of of the "robot from" on the array of strings.

Docs updates are coming in a subsequent PR from the book-sql branch.

This commit changes the "robot from" syntax from the eval() structure
to a simple f-string.  This recognizes that entities in from operators
are always identified as strings.  The ability to scan the entities in
an array of strings is no longer available with this change, but this can
always be emulated by doing an "over" upstream of of the "robot from"
on the array of strings.

Docs updates are coming in a subsequent PR from the book-sql branch.
Copy link
Member

@nwt nwt left a comment

Choose a reason for hiding this comment

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

This is good! :shipit:

@mccanne mccanne merged commit a3819d1 into main Dec 16, 2025
4 checks passed
@mccanne mccanne deleted the from-fstring branch December 16, 2025 19:07
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