I feel the documentation for the :headers option while parsing is confusing.
It states:
:headers - when false, no longer discard the first row. Defaults to true.
I had to test the option to see exactly how it works, and it seems to be the opposite of what I'd expect -- setting headers: trueskips the headers row, and so I'd expect it to be called e.g. skip_headers.
It's easy enough to update the documentation, but what are your thoughts about deprecating :headers and changing it to :skip_headers?
I feel the documentation for the
:headersoption while parsing is confusing.It states:
I had to test the option to see exactly how it works, and it seems to be the opposite of what I'd expect -- setting
headers: trueskips the headers row, and so I'd expect it to be called e.g.skip_headers.It's easy enough to update the documentation, but what are your thoughts about deprecating
:headersand changing it to:skip_headers?