search

Data-Liberation-Front / csvlint.rb

The gem behind http://csvlint.io
MIT License
283 stars 86 forks source link

eliminate some date and time formats, related #105 #119

Closed jpmckinney closed 9 years ago

jpmckinney commented 9 years ago

From #105

Several date and time formats were blindly imported from Rails' Date/Time formats, but I'd expect several of those are never in a CSV (e.g. the 23 digit timestamp with nanoseconds).

I've removed the long numbers-only date/time formats, because these can false positive, e.g. if you have 6-digit identifiers, a lot of them will look like dates, e.g. 12340101.

There are very likely other formats that can be removed due to their never appearing in CSVs.

Fixes #106

coveralls commented 9 years ago

Coverage Status

Coverage increased (+0.51%) to 96.42% when pulling 1fe84b19dd14103be68d97a26075c4bc3f668533 on jpmckinney:fewer_formats into 08ab3af84a892ab3eed054d8f60da1926e5ec67f on theodi:master.