-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support lower case keywords #31
Comments
I thought about doing this as well in the PR that fixed the EOI rule. It's a simple |
In my case, I have By supporting lower-case keywords, we still allow people to differentiate between keywords and identifier using upper-case/lower-case words, but we support those who prefer a smoother typing experience as well. |
There's a problem with this. "from" is also included in the tx_field_list, so it parses "from" as fields instead of FROM ~ entity
any suggestion how to solve this? maybe we can use a special case for this? like how postgres does it (using double quotes, a block "[ ]") or changing the 'from' field name |
@rizdarmwn I'm thinking about using another keyword to replace |
hmm, I see. but that will trip users off when using it. expectations was it's SQL for the EVM networks. i think we can just use the special case for the "from" fields, no? when using lowercases |
I'm not a big fan of the special characters approach. I'd like the syntax to look as clean as possible, and that's a step in the other direction.
The "SQL for EVM networks" is just a way to quickly communicate what EQL is, it shouldn't mean that we need to follow the language specs. We've already introduced |
Sure, then I'll try to change it to |
Describe the solution you'd like
At the moment EQL only supports upper-case keywords like
GET, FROM, ON
, and that can be annoying for someone that's looking for speed instead of differentiating between keywords and other elementsAdditional context
The solution is simply ignoring case for the keyword defined on production.pest
The text was updated successfully, but these errors were encountered: