Skip to content
This repository has been archived by the owner on Jul 15, 2021. It is now read-only.

Feature Request: Add location data to AST #31

Open
jdrew1303 opened this issue Oct 24, 2016 · 1 comment
Open

Feature Request: Add location data to AST #31

jdrew1303 opened this issue Oct 24, 2016 · 1 comment

Comments

@jdrew1303
Copy link

Adding location data to nodes in the AST would allow for 'possibly' new languages to be built using the AST as a target (stage 2 of my master plan, think SASS for SQL 😉 ). (This is similar to how ES6 -> ES5 transpilers work, and indeed Java, Scala, etc).

The parser in this case could be thought of as a reference spec for an SQL AST.

@DanielSWolf
Copy link

I'd like to use the parser as basis for a linter. But without location data, there is no way of knowing where a certain AST element originated in the query.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants