-
Notifications
You must be signed in to change notification settings - Fork 111
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
Support TOML 1.1 release #397
Comments
Note: this is blocked on the release being out but meant for our tracking of it |
Has there been any progress on the inline-table with multiline support? I looked at some of the history but couldn't find anything yet. I think it's a nice usability improvement in certain cases. We might be able to spend some time at @prefix-dev if the feature is desired. |
At this time, I'd rather not be supporting unreleased TOML specs. The issue for TOML 1.1 spec is toml-lang/toml#928 |
And if we would put it behind a feature flag like |
That could get quite messy maintaining different subsets of the parsers like that. We'd also need a good way of dividing the conformance tests to handle that. I would also make it a Overall, this does not seem worth doing, especially because its not guaranteed that the approved parts of the format will stay as-is |
atm the next toml release contains
The unicode one will be a challenge as it might require us to stop using byte parsing and switch back to UTF-8 parsing which would slow us down a good chunk (#219). Its not clear if they reused any existing unicode definitions like xid_continue, so I asked in toml-lang/toml#891
The text was updated successfully, but these errors were encountered: