DataJunction / dj

A metrics platform.
http://datajunction.io
MIT License
34 stars 15 forks source link

Parse contexts that need to be handled by the DJ AST visitor #420

Open samredai opened 1 year ago

samredai commented 1 year ago

There are a few remaining parse contexts that still need to be handled by the DJ AST visitor. Adding them here so we don't lose track of these.

The following are probably things we won't allow in node queries so we just need to catch them and raise a clear error:

shangyian commented 1 year ago

FYI -- just added support for BooleanLiteralContext while in the middle of getting the type inference tests to work.

shangyian commented 1 year ago

Added SubscriptContext. This is a lot cleaner with the new ast!