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

Documentation #4

Merged
merged 24 commits into from
Jan 23, 2025
Merged

Documentation #4

merged 24 commits into from
Jan 23, 2025

Conversation

anitaapplegarth
Copy link
Contributor

Issue

  • Which issue does your pull request address? If none of the existing issues are applicable, you can write a new issue.

Testing, documentation, and checks

  • If your pull request introduces new code, you should write automated tests (explained here).
  • Check that all tests pass using devtools::check().
  • Check compliance with the preferred R style using lintr::lint_package().
  • Build documentation using devtools::document().
  • If you changed the readme, run knitr::knit("README.Rmd").

Pull request

  • Assign at least one reviewer.

@anitaapplegarth
Copy link
Contributor Author

Please check it makes sense and note any edits you think need to be made. This is not the final version but want to make sure this is on the right track.

Changes suggested by Matthew agreed

Co-authored-by: mghosh00 <[email protected]>
Copy link
Contributor

@mghosh00 mghosh00 left a comment

Choose a reason for hiding this comment

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

Looks great!!

@anitaapplegarth anitaapplegarth merged commit be84156 into main Jan 23, 2025
10 checks passed
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