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

Ron review doc #109

Merged
merged 5 commits into from
Feb 24, 2025
Merged

Ron review doc #109

merged 5 commits into from
Feb 24, 2025

Conversation

rklasky
Copy link
Collaborator

@rklasky rklasky commented Feb 7, 2025

Rpath_support.R

  • Updated title, description, and examples tags for all functions
  • Changed return tags to returns
  • Added \code tags
  • Fixed bug in rpath.mixotrophs. Code should check types between 0 and 1 and not between 0 and 0.

ecosim_plot.R

  • Added section tag to point to vignette which shows example usage

@rklasky rklasky assigned kaydin and andybeet and unassigned kaydin and andybeet Feb 7, 2025
@rklasky rklasky requested review from andybeet and kaydin February 7, 2025 20:34
Copy link
Member

@andybeet andybeet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I fixed an issue with the NAMESPACE file that was preventing building. All looks good.
The crosslinks/autolinks might not work until we update a few thinks mentioned in this discussion.

@rklasky
Copy link
Collaborator Author

rklasky commented Feb 9, 2025

Added \code tag to vignette link in ecosim_plot.R

@kaydin kaydin merged commit cdc8650 into docs_pre_release Feb 24, 2025
2 checks passed
@andybeet andybeet mentioned this pull request Feb 25, 2025
15 tasks
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.

3 participants