-
Notifications
You must be signed in to change notification settings - Fork 13
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
About JSON interface #21
Comments
Yeah, I was still working on it. And the PR was just to show what I was doing. Not sure why @data-man merged it but we should probably undo that until it's done properly. |
Within the PR we would not receive feedback. :) |
How does JSON handle this? Wouldn't it have the same issue? |
@PMunch yeah, with JSON I would do it the same way, but with older API of this lib it was easier :) But yeah, it's fine for me |
I'm considering splitting this into multiple files. One with the object definition and parsing/converting stuff, and others with interface logic. This way we can actually have different interfaces. |
I like it, but it has one big drawback, code, which previously was written like:
where config is a section of a TOML file
now needs to be written like:
The text was updated successfully, but these errors were encountered: