Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: James Azam <james.azam@lshtm.ac.uk>
  • Loading branch information
sbfnk and jamesmbaazam authored Sep 20, 2024
1 parent 20451cc commit 46793c0
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion NEWS.md
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@

## Package changes

- `fix_dist()` has been renamed to `fix_parameters()`. By @sbfnk in #733 and reviewed by @jamesmbaazam.
- `fix_dist()` has been renamed to `fix_parameters()` because it removes the uncertainty in a distribution's parameters. By @sbfnk in #733 and reviewed by @jamesmbaazam.

## Bug fixes

Expand Down
4 changes: 2 additions & 2 deletions R/deprecated.R
Original file line number Diff line number Diff line change
Expand Up @@ -624,10 +624,10 @@ apply_tolerance <- function(x, tolerance) {
return(y)
}

#' Fix the parameters of a `<dist_spec>`
#' Remove uncertainty in the parameters of a `<dist_spec>`
#'
#' @description `r lifecycle::badge("deprecated")`
#' This function has been renamed to [fix_parameters()].
#' This function has been renamed to [fix_parameters()] as the former was a misnomer.

Check warning on line 630 in R/deprecated.R

View workflow job for this annotation

GitHub Actions / lint-changed-files

file=R/deprecated.R,line=630,col=81,[line_length_linter] Lines should not be more than 80 characters. This line is 85 characters.
#' @return A `<dist_spec>` object without uncertainty
#' @keywords internal
#' @importFrom cli cli_abort
Expand Down

0 comments on commit 46793c0

Please sign in to comment.