Erratum: Mismatch on default for `commentPrefix`

Question

Erratum: Mismatch on default for `commentPrefix`

Opened this issue 9 years ago · 3 comments

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

Answer 1 · 2016-02-02T17:00:02.000Z

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.

Answer 2 · 2016-02-02T17:07:36.000Z

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.

Answer 3 · 2016-02-03T15:04:48.000Z

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