Erratum: Mismatch on default for `commentPrefix`
Opened this issue · 3 comments
JeniT commented
In the metadata document it says #
; in the model document it says null
. I think it should be null
.
gkellogg commented
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
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
Summary: The decision on the call, 2016-02-03 is that the metadata document should say null