csvside

csvside copied to clipboard

support case-insensitive headers

2

Hi there I notice when there is different cases on the headers, the parsing will fail. For example replace `RowReaderFixtures.testReader` line ```"Str".read[String],``` with this line ```"str".read[String],``` and you will have...

Andrea

jonoabroad

Current implementation does not allow configuration of underlying CSVReader, such as separator character and quote character. Simple implementation could have an implicit `CSVReaderConfiguration` implementation.

gedejong

In the README we use: ``` import cats.syntax.cartesian._ ``` This has been deprecated. Let's update the README.

d6y

[Bug] Cannot parse empty cells

1

Current implementation does not allow parsing empty cells: @ ``` Invalid(List(CsvError(2,CsvPath(List(Naam2)),Column was empty))) ``` even when typing the case class with `Option[String]`. #

gedejong

Using `fromFile` generates an exception with a stream closed too soon. The iterator is lazy, and we close the file around it before a program can consume the rows.

d6y

A CSV with duplicate column names... ``` Home School, Allocation, Joint School, Allocation ``` ...modelled as: `(String, Double, Option[String], Option[Double])` will fail to parse a valid row such as: ```...

d6y