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
Use Enums for the AST #134
Comments
@chaosprint I'd love to take a shot at this, if you're not planning on doing it soon. I also think this could be useful in implementing more robust docs, as all Node types could be in the same module and their doc comments could state how they should be used. (I also think it would be cool to make this documentation available in glicol-cli; not quite certain how that would work but I think it would be cool to have auto-generated information available inside glicol-cli about exactly what it supports) |
Sure, feel free to take it on. For the cli, perhaps we can use https://github.com/mikaelmello/inquire in the future so that when users type
|
So I'm still working on this (I've got the ast parsing in Do we want to make sure the AST can be easily programmatically created (i.e. make sure someone can create an AST without just parsing a snippet of code), or is that not a use-case we want this library to support? It wouldn't be too hard to support that (we could just make all the string borrows in the Ast enum be a What are your thoughts? (Also, did I explain this well? If you have questions, obviously feel free to ask) |
Sorry for the late reply. I agree with you. There's no need to worry about making ASTs manually. If we're programming sounds instead of using Glicol syntax, we can just use what's in Can't wait to see the zero-copy stuff! |
No description provided.
The text was updated successfully, but these errors were encountered: