-
Notifications
You must be signed in to change notification settings - Fork 4
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
Fix: remove epitrix dependency and add clean_label function #219
Conversation
This pull request:
(Note that results may be inaccurate if you branched from an outdated version of the target branch.) |
This pull request:
(Note that results may be inaccurate if you branched from an outdated version of the target branch.) |
This pull request:
(Note that results may be inaccurate if you branched from an outdated version of the target branch.) |
This pull request:
(Note that results may be inaccurate if you branched from an outdated version of the target branch.) |
R/cleaning_data.R
Outdated
#'} | ||
#' @note El código original fue escrito por los autores de \pkg{epitrix}. | ||
#' Consulta \url{https://CRAN.R-project.org/package=epitrix} para más detalles. | ||
#' @keywords internal |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would also probably add @noRd
here so that no help file is created and this doesn't cause issues with CRAN.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
DESCRIPTION
Outdated
@@ -45,7 +45,7 @@ RoxygenNote: 7.3.2 | |||
Imports: | |||
config, | |||
dplyr, | |||
epitrix, | |||
stringi, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note that the other dependencies are listed alphabetically. You'll need to move this down if you want to keep with this convention.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was resolved in commit: c3483e2
This pull request:
(Note that results may be inaccurate if you branched from an outdated version of the target branch.) |
This pull request:
(Note that results may be inaccurate if you branched from an outdated version of the target branch.) |
This PR includes: