-
Notifications
You must be signed in to change notification settings - Fork 36
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
High-level stable API #56
Comments
Yep I think things are fairly settled down now in terms of the backend. What sort access functions are you after? |
Some sort of API where I don't have to access any internals. For an Basically, if you change the internals to be completely heterogeneous (no more |
Alright will do, I'll draw up a doc outlining proposed getters in the style of the current |
It'd also be good to think about the overlay tree stuff we'd discussed (created on demand with stored parents and absolute locations) and probably put it in this package. Deprecations.jl has an ad hoc version of it, but it's probably not the right approach. |
I'm in the progress of upgrading FemtoCleaner (the GitHub bot that fixes deprecations) from CSTParser 0.2.1 to the current version of CSTParser.
However, progress is very slow because since CSTParser does not have an API, a lot of the code in FemtoCleaner depends on the internals of CSTParser, which are now completely changed.
If the idea is to use CSTParser to build nice tools, I feel there needs to be a more high-level API where some stability is expected.
The text was updated successfully, but these errors were encountered: