leeper/rio

…e failing to install, from upstream: Rdatatable/data.table#3938

* Added dynamic argument checking for 'readODS:read_ods()' as proposed in #223. Also more detailed tests.

* '.import.rio_xls()' no longer passing '...' to 'read_xls()' because that functions cannot accept '...' arguments. Instead, the length of '...' is checked and if > 0 a warning is issued. With corresponding tests. Also went back and changed my earlier 'read_xlsx()' test to be 'expect_warning()' instead of 'expect_true()' since by that point that's what needs testing.

* Add support for pzfx files

* Completion of .pzfx addition by documenting in NEWS.md

* Give message if pzfx test is skipped due to missing library

* Remove pzfx as import requirement

…to 'load()' nor to 'readRDS()' because those underlying functions cannot accept '...' arguments. Instead, the length of '...' is checked and if > 0 a warning is issued. With corresponding tests. As proposed in #223 (#229)

…opefully I guessed the project settings correctly from the existing files (#227)

…t as per #222 (review) . Please note my response in that thread to the last question about the 'rows' argument.

* Removed duplicated example line in import.R

* Run devtools::document() to propagate change to import.Rd.

…xlsx or read.xlsx and discards all arguments in '...' that don't match, instead of having to catch them individually. Included tests. While I was at it, noticed there were no xls tests, so added a test file and tests (but left .import.rio_xls untouched so far)

* Add lines to extensions.R for plaintext accounting formats supported by ledger package

* Add tests and fix bmp import typo

* Just a simple expect_error check