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

Accidentally repeated roxygen comments #603

Closed
remlapmot opened this issue Nov 22, 2023 · 3 comments · Fixed by #604
Closed

Accidentally repeated roxygen comments #603

remlapmot opened this issue Nov 22, 2023 · 3 comments · Fixed by #604
Labels

Comments

@remlapmot
Copy link

Thanks for such a great package.

When reading some of the helpfiles, I noticed that there are accidentally repeated roxygen comments in 6 places in the code:

#' #'Van Buuren, S. (2018).

#' #'Van Buuren, S. (2018).

#' #'Van Buuren, S. (2018).

mice/R/mice.R

Line 113 in 3abe393

#' #'A new argument \code{ls.meth} can be parsed to the lower level

mice/R/mira.R

Line 35 in 3abe393

#' #'The \code{mira} class of objects has methods for the

#' #'Van Buuren, S. (2018).

i.e., the second #' just needs to be deleted in each case.

@remlapmot remlapmot added the bug label Nov 22, 2023
@hanneoberman hanneoberman linked a pull request Nov 23, 2023 that will close this issue
@hanneoberman
Copy link
Member

hanneoberman commented Nov 23, 2023

Thanks! I've created a PR with your suggested changes #604

@remlapmot
Copy link
Author

Thank you very much. Maybe you need to run devtools::document()/roxygen2::roxygenize() to regenerate the .Rd files and add those to the PR also.

@hanneoberman
Copy link
Member

You're absolutely right! I thought I could just fix it in the online repo, but running it locally now. Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants