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.

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

Update documentation for new compare app #58

Closed
lmsimp opened this issue Jan 28, 2016 · 3 comments
Closed

Update documentation for new compare app #58

lmsimp opened this issue Jan 28, 2016 · 3 comments

Comments

@lmsimp
Copy link
Collaborator

lmsimp commented Jan 28, 2016

See also #33
@lmsimp
Copy link
Collaborator Author

lmsimp commented Jan 29, 2016

@lgatto I am having trouble re-generating the man file using Roxygen.

  • I have renamed the old pRolocVis_compare app pRolocVis_compare_legacy
  • The new compare app is now found in pRolocVis_compare and I have added @rdname pRolocVis-apps tag to the code so that the pRolocVis wrapper finds it.
  • I generate the man file using the following code R --vanilla -e "library(roxygen2); roxygenize('pRolocGUI', roclets='rd')" but the documentation does not seem to get updated?

What am I doing wrong? Would you mind taking a look at the branch? I haven't merged yet, I want to make sure all docs are correct and am still updating the vignette. Sorry for the delay.

@lgatto
Copy link
Owner

lgatto commented Jan 29, 2016

That all looks correct. I have added a missing @rdname in pRolocVis_legacy.R.

@lgatto
Copy link
Owner

lgatto commented Jan 30, 2016

@lmsimp I have merge compare-app into master.

@lmsimp lmsimp closed this as completed Feb 3, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@lgatto @lmsimp