-
Notifications
You must be signed in to change notification settings - Fork 5
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
added nonNegativeInteger #35
Conversation
Ah right. Of course. Will do! And I will leave that topic branch open for now anyway, perhaps there'll be more types. |
OK, I've fixed the test, but it's not clear to me whether it's correct what I'm doing here. I'm currently raising a |
@stschiff I just remembered why I didn't implement {
"name": "...",
"datatype": {
"base": "integer",
"minimum": 0
}
} |
Codecov Report
@@ Coverage Diff @@
## master #35 +/- ##
=======================================
Coverage 99.90% 99.90%
=======================================
Files 15 15
Lines 2211 2222 +11
=======================================
+ Hits 2209 2220 +11
Misses 2 2
Continue to review full report at Codecov.
|
Yes, that makes sense, I also realised as much. I'm open to doing it this way, it would require some minor changes in my schemas, but nothing dramatic. Up to you. The official csvw standard has all these edge-case datatypes, in principle, so perhaps it's good to support them even if they're somewhat redundant. |
Yes, I think we should support as many as possible, in particular if its not much work (o the work is already done :) ). |
I'm not a big fan of topic branches, though, or any other kind of long-lived branch - so I'll merge now :) |
just added this one new datatype, perhaps more to come if need be.