-
Notifications
You must be signed in to change notification settings - Fork 42
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
Lists #15
Comments
I can't remember but I guess that's because you can use a "reviver" and parse the array with |
Now I remember... This syntax is not supported. This module is a .properties parser/stringifier which can be tweaked a bit, but parsing arrays like that is not supported and never will be. |
I'm reconsidering this. |
Given a JS object like this: {
foobar: [
1,
2,
{
key: 'value'
}
]
} This could easily be represented as:
As well, when parsing the file, one could do either: {
foobar: [1,2, {key: 'value'}]
} or {
foobar: {
0: 1,
1: 2,
2: { key: 'value' }
}
} And leave it up to the end user to decide which format, without having to write their own code to do either format. https://github.com/alex-dow/icu-converter/blob/develop/src/writers/properties.js I've written a basic JS object to properties file serializer there, but it is no where near as robust as this project, and I would to see this project handle nested datatypes and would be happy to help. |
+1 |
Do one thing and do it good. Why not to stick to default |
This is more of question than and issue. Why did you choose not to support arrays of objects?
ex.
From what I can tell that would have to be:
properties file:
output:
Did I miss something?
The text was updated successfully, but these errors were encountered: