Skip to content
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.

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

Updating some of the documentation #1

Merged
merged 3 commits into from
Jul 2, 2020
Merged

Conversation

dewittpe
Copy link
Contributor

@dewittpe dewittpe commented Jul 1, 2020

These changes update most of the documentation. There is more to do, additional issues are going to be filed to document additional work to be done, but these changes were quick and easy and will be a good starting point.

Prior to this change the initial definition for the esteq documentation
had the tag `@name est_eq` but all the functions used `@rdname esteq`.
The result was inconsistent documentation and missing references in the
NAMESPCE.

This change chagnes the roxygen comment to `@name esteq` and updates all
documentation files.
Prior to this change Y1 was documented as functional arguement but not
part of the function definition.  R CMD check identified Y1 as a
undefined global variable as a result.  I suspect no errors were thrown
during testing because the any use of Y1 in a call would be captured by
the `...` argument.

This change add Y1 explicitly to the function arguments.
This was referenced Jul 1, 2020
@kevjosey kevjosey merged commit 68e3a29 into kevjosey:master Jul 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants