sqlite-parser icon indicating copy to clipboard operation
sqlite-parser copied to clipboard

Node Structure Change: Change variant names to camelCase and change 'function' variant name

Open jdrew1303 opened this issue 8 years ago • 1 comments

image

When switching on type you need to use strings because of the whitespace in some of the names. This isn't too important but would be a nice to have.

Also the use of function as the variant type could cause issues for people building tooling on top of it. Again nothing major.

jdrew1303 avatar Oct 24 '16 15:10 jdrew1303

I think the AST should move to PascalCase or camelCase for the values of these properties but it might have to wait until release v2.0.0.

nwronski avatar Oct 30 '16 23:10 nwronski