Skip to content

Refactor: satisfy CRAN requirements and reduce dependencies#54

Merged
martinctc merged 6 commits into
mainfrom
refactor/changes-for-cran
Feb 3, 2021
Merged

Refactor: satisfy CRAN requirements and reduce dependencies#54
martinctc merged 6 commits into
mainfrom
refactor/changes-for-cran

Conversation

@martinctc

@martinctc martinctc commented Jan 22, 2021

Copy link
Copy Markdown
Member

Summary

This branch implements a series of changes to satisfy CRAN requirements. This includes reducing the number of dependencies. The merging of this PR will wrap up version 1.3.1, and the next PR merge should integrate the package version to 1.3.2.9000 or 1.4.0.

Changes

The changes made in this PR are:

  1. Removed package dependencies that appear on DESCRIPTION but are not explicitly used in functions.
  2. Use direct links in README.md (as per CRAN submission feedback).
  3. Clean up .Rbuildignore.
  4. Clean up doc to update vignette files in the folder.
  5. Add trailing slashes to URLs, and remove re-directing URLs.

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.

A series of changes to satisfy CRAN requirements. This includes reducing the number of dependencies
Also change way of adding lifecycle badges to comply with CRAN
Part of CRAN compliance
@martinctc martinctc marked this pull request as ready for review January 25, 2021 17:50
@martinctc martinctc self-assigned this Jan 25, 2021
Changes include:
- Trailing slashes now added
- Redirecting
  URLs replaced
- Check for http URLs (none
  found)
@martinctc martinctc merged commit 786d97d into main Feb 3, 2021
@martinctc martinctc deleted the refactor/changes-for-cran branch February 3, 2021 11:32
@martinctc martinctc mentioned this pull request Feb 26, 2021
2 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.

1 participant