w3c / csvw

Documents produced by the CSV on the Web Working Group
Other
162 stars 57 forks source link

Erratum: Mismatch on default for `commentPrefix` #801

Open JeniT opened 8 years ago

JeniT commented 8 years ago

In the metadata document it says #; in the model document it says null. I think it should be null.

gkellogg commented 8 years ago

My implementation uses "#" as the default value, but there are no tests to verify that it is used this way; probably because it only comes in when we're skipping header rows. I'm fine with making such a change.

iherman commented 8 years ago

If we agree on changing the metadata document in this respect, than we should remove the label on the model before acknowledging the erratum. Otherwise I am fine with it.

iherman commented 8 years ago

Summary: The decision on the call, 2016-02-03 is that the metadata document should say null