malecki/apsrtable — pull request #5 by jknowles (2014-10-21 16:03:16+00:00)
Title: Make dev branch pass CRAN

This PR should close #3 and #1 entirely. It revises the documentation in the dev branch to be compatible with roxygen, eliminates all of the legacy .Rd files in favor of auto-generated .Rd files, allows the `roxygen` package to handle the NAMESPACE, collate, and all of the generation of documentation.

It also cleans up the `modelInfo` method substantially. This will make it easier to add new `modelInfo` to this package as they become available, although more work is needed to make clear what needs to be passed out of a `modelInfo` object.

This PR also eliminates any CRAN warnings that resulted from duplicated documentation, unexported global variables, issues with undocumented paramaters.

It also cleans up the `apsrtableSummary` documentation to use `object` instead of the more generic `x` as the main parameter. This is slightly more clear to the user and will not affect any legacy code.

It also takes a small step forward in building a test suite to ensure future changes do not break the ability to produce tables. This work should be expanded on in another branch.

Take a look @malecki and let me know what you think -- it is a substantial revision, but I think everything is working the way it was before and the code/documentation is more tightly integrated which should make future modifications easier.