define/2 should have a list as default second parameter.

dashbitco / nimble_csv

A simple and fast CSV parsing and dumping library for Elixir

https://hexdocs.pm/nimble_csv

767 stars 51 forks source link

define/2 should have a list as default second parameter. #54

Closed fcevado closed 4 years ago

fcevado commented 4 years ago

it's pretty weird that i need to pass an empty list when i just want the default options to work. I already have the changes for that, if you agree, i'll open the PR.