Deprecate release() in favour of usethis::use_release_issue()

Fixes #2500

Author: hadley

NEWS.md

@@ -4,6 +4,7 @@
 * New `check_mac_devel()` function to check a package using the macOS builder at https://mac.r-project.org/macbuilder/submit.html (@nfrerebeau, #2507)
 * `is_loading()` is now re-exported from pkgload (#2556).
 * `load_all()` now errors if called recursively, i.e. if you accidentally include a `load_all()` call in one of your R source files (#2617).
+* `release()` is deprecated in favour of `usethis::use_release_issue()`.
 * `show_news()` now looks for NEWS files in the same locations as `utils::news()`: `inst/NEWS.Rd`, `NEWS.md`, `NEWS`, and `inst/NEWS` (@arcresu, #2499).
 
 # devtools 2.4.6

R/release.R

@@ -1,23 +1,12 @@
 #' Release package to CRAN.
 #'
-#' Run automated and manual tests, then post package to CRAN.
-#'
-#' The package release process will:
-#'
-#'   * Confirm that the package passes `R CMD check` on relevant platforms
-#'   * Confirm that important files are up-to-date
-#'   * Build the package
-#'   * Submit the package to CRAN, using comments in "cran-comments.md"
-#'
-#' You can add arbitrary extra questions by defining an (un-exported) function
-#' called `release_questions()` that returns a character vector
-#' of additional questions to ask.
+#' @description
+#' `r lifecycle::badge("deprecated")`
 #'
-#' You also need to read the CRAN repository policy at
-#' 'https://cran.r-project.org/web/packages/policies.html' and make
-#' sure you're in line with the policies. `release` tries to automate as
-#' many of polices as possible, but it's impossible to be completely
-#' comprehensive, and they do change in between releases of devtools.
+#' `release()` is deprecated in favour of [usethis::use_release_issue()].
+#' We no longer feel confident recommrding `release()` because we don't use it
+#' ourselves, so there's no guarantee that it will track best practices as
+#' they evolve over time.
 #'
 #' @template devtools
 #' @param check if `TRUE`, run checking, otherwise omit it.  This
@@ -29,6 +18,11 @@
 #'   tasks that you can use in addition to or in place of `release`.
 #' @export
 release <- function(pkg = ".", check = FALSE, args = NULL) {
+  lifecycle::deprecate_warn(
+    "2.5.0",
+    "release()",
+    "usethis::use_release_issue()"
+  )
   pkg <- as.package(pkg)
   # Figure out if this is a new package
   cran_version <- cran_pkg_version(pkg$package)
@@ -159,6 +153,10 @@ find_release_questions <- function(pkg = ".") {
 }
 
 yesno <- function(msg, .envir = parent.frame()) {
+  if (!rlang::is_interactive()) {
+    cli::cli_abort("Called from non-interactive context.")
+  }
+
   yeses <- c(
     "Yes",
     "Definitely",

_pkgdown.yml

@@ -26,7 +26,6 @@ reference:
   - document
   - load_all
   - reload
-  - release
   - spell_check
   - test
 - title: Package Installation
@@ -49,6 +48,10 @@ reference:
   - wd
   - save_all
 
+- title: Deprecated functions
+  contents:
+  - release
+
 news:
   releases:
   - text: "Version 2.2.1"

man/release.Rd

@@ -0,0 +1,11 @@
+# release() is deprecated
+
+    Code
+      . <- release()
+    Condition
+      Warning:
+      `release()` was deprecated in devtools 2.5.0.
+      i Please use `usethis::use_release_issue()` instead.
+      Error in `yesno()`:
+      ! Called from non-interactive context.
+

tests/testthat/_snaps/release.md

@@ -0,0 +1,3 @@
+test_that("release() is deprecated", {
+  expect_snapshot(. <- release(), error = TRUE)
+})