New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Final CRAN prep #134

Merged

lukesonnet merged 3 commits into master from lukesonnet/cran_final

Feb 15, 2018

Contributor

lukesonnet commented Feb 15, 2018 •

edited

Remove vignette suggest package
spell check
styler pass
Passes devtools::check() (no E, W, N)
Passes R CMD check locally (no E, W, N)
Passes all CI tools, only NOTEs are about package size on linux and about availability of enhance packages on appveyor

* checking package dependencies ... NOTE
Packages which this enhances but not available for checking:
  'broom' 'texreg'
* checking Rd cross-references ... NOTE
Package unavailable to check Rd xrefs: 'texreg'

lukesonnet added 3 commits

February 14, 2018 16:50


          mostly whitespace styler edits

f021e74


          remove vignette dependencies

57a8dec


          spelling check

9eae931

nfultz approved these changes

View reviewed changes

lukesonnet merged commit e92cdbd into master

lukesonnet deleted the lukesonnet/cran_final branch

February 15, 2018 01:33

coveralls commented Feb 15, 2018

Coverage increased (+0.0008%) to 99.634% when pulling 9eae931 on lukesonnet/cran_final into bdcbc7f on master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment