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

Pull changes from main into branch #22

Closed
wants to merge 7 commits into from
Closed

Conversation

stephaniereinders
Copy link
Member

No description provided.

stephaniereinders and others added 7 commits August 20, 2024 15:13
Addressed the following issues documented by CRAN:

Please always write package names, software names and API (application
programming interface) names in single quotes in title and description.
e.g: --> 'shiny'
Please note that package names are case sensitive.

Please omit the redundant "in R" at the end of your title.

Please use only undirected quotation marks in the description text.
e.g. `handwriterApp` --> 'handwriterApp'

Please add \value to .Rd files regarding exported methods and explain
the functions results in the documentation. Please write about the
structure of the output (class) and also what the output means. (If a
function does not return a value, please document that too, e.g.
\value{No return value, called for side effects} or similar)
  Missing Rd-tags:
      handwriterApp.Rd: \value
- Two of the model_docs still had the original 300 DPI. I changed them to 72 DPI in photoshop. I also made this change in tests > testthat > template.
- The template_docs folder in extdata > template isn't used by the app so I deleted it.
Fixed issues flagged by CRAN
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.

1 participant