-
Notifications
You must be signed in to change notification settings - Fork 1
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
README is unclear #24
Comments
There's already tons of examples in the README, what specifically do you think should be added? |
You could for example document the way this works through use cases from command line. {
"type": "record",
"name": "options_test_record",
"fields": [
{
"name": "myUuid",
"type": {
"type": "string",
"arg.properties": {
"regex": "[0-9A-F]{8}-[0-9A-F]{4}-4[0-9A-F]{3}-[89AB][0-9A-F]{3}-[0-9A-F]{12}"
}
}
},
{
"name": "myEnumType",
"type": {
"type": "enum",
"name": "enum_test",
"symbols": [
"MY_VALUE_1",
"MY_VALUE_2",
"MY_VALUE_3"
],
"arg.properties":
{
"options": [
"MY_VALUE_1",
"MY_VALUE_2",
"MY_VALUE_3"
]
}
}
},
{
"name": "myNumber",
"type": {
"type": "int",
"arg.properties": {
"range" : {"min": 1, "max": 10}
}
}
}
]
} May be I missused the enum type but I was surprised to see I had to add both 'symbols' and 'options' fields. Does it make sense ? |
Ah, CLI examples could be useful. With regards to the enum example--are you familiar with Avro at all? Because that's a bit of a prerequisite for using this tool. We also clearly demonstrate here that you do not need to specify a value for |
I did some Avro yes.
|
I think that the readme misses some examples to understand the correct syntaxes.
It would be helpfull to give example for each flags and the corresponding schema used for each use case.
Thanks !
The text was updated successfully, but these errors were encountered: