adam3smith / dataverse-client-r

Currently fq is completely broken:
1. It requires start to be set for no reason
2. It fails because match.arg will always fail since no args are given in the function

This fix does three things:
1. Test for fq rather than start
2. get rid of the match.arg -- since fq is super flexible, we can't use that here
3. enclose the fq query in I to prevent URL encoding (see r-lib/httr#540 (comment) ). Otherwise, httr encodes colons, square brackets and + and e.g. the [example for date range search](http://guides.dataverse.org/en/latest/api/search.html#date-range-search-example) breaks

replace 'ghit' package with 'remotes'

ref IQSS#24 & IQSS#25

ref IQSS#25

since knitr is already a 'suggest'ed package, and rmarkdown is a dependency (https://CRAN.R-project.org/package=knitr ).

ref IQSS#25

ref IQSS#25

(currently Travis's default is Xenial)

ref IQSS#25

and change `R` (in master) to `Rscript` in the `after_success` block

ref IQSS#25

`usethis::use_coverage()` and other adivce from https://github.com/r-lib/covr

ref IQSS#25

and use the default notifications, which are mostly what was specified: https://docs.travis-ci.com/user/notifications/#default-notification-settings

ref IQSS#25

ref IQSS#25

https://travis-ci.org/IQSS/dataverse-client-r/jobs/606549550#L1738

https://travis-ci.org/IQSS/dataverse-client-r/jobs/606549550#L1739

The `after_success` block is still Different.  I go the `r_packages` block confused with  `r_github_packages`

Some rules & recommendations have changed.  https://github.com/r-lib/covr#codecov

ref IQSS#24

It was removed from CRAN May 2018.  The 'remotes' package has the same functionality (and function names)

to avoid a Roxygen note when checking package

from 6.1.1.  There appears to be a few formatting changes in the autogenerated Rd file