Skip to content

Conversation

@martinctc
Copy link
Member

@martinctc martinctc commented Feb 5, 2021

Summary

This branch implements some changes mostly to the documentation, for CRAN compliance purposes.

Changes

The changes made in this PR are:

  1. Add Microsoft Corporation as a copyright holder in the Authors@R field in DESCRIPTION.
  2. Update package start-up message to include package version number.
  3. Improve documentation on validation_report(), including the constituent checking functions.
  4. More details added to the new working patterns functions introduced in 1.4.0.
  5. Values returned are specified clearly for all exported functions.
  6. Copyright headers are added back in where they were missing.

Checks

  • All R CMD checks pass
  • roxygen2::roxygenise() has been run prior to merging to ensure that .Rd and NAMESPACE files are up to date.

Notes

This PR partially fixes the below issues. Many are blanket changes to a lot of the documentation in the package, and therefore would take a longer time to fully implement:

@martinctc martinctc self-assigned this Feb 5, 2021
@martinctc martinctc added the documentation Improvements or additions to documentation label Feb 5, 2021
martinctc added a commit that referenced this pull request Feb 5, 2021
This is directly committed to the main branch to test the experience on the GitHub site. This is part of a series of changes linked to #61.
@martinctc martinctc marked this pull request as ready for review February 9, 2021 22:21
@martinctc martinctc merged commit 4228be4 into main Feb 9, 2021
@martinctc martinctc deleted the docs/cran-compliance branch February 9, 2021 22:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants