Allow multiple emails for authors (#1529)

Fixes #1487
r-lib · Nov 6, 2023 · 3d162cb · 3d162cb
1 parent a6ab093
commit 3d162cb
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 1 deletion.
diff --git a/NEWS.md b/NEWS.md
@@ -1,5 +1,7 @@
 # roxygen2 (development version)
 
+* authors in `DESCRIPTION` can now have multiple email addresses (@jmbarbone, #1487).
+
 * The `ROXYGEN_PKG` environment variable is now set up while roxygen
   is running to the name of the package being documented (#1517).
 

diff --git a/R/object-package.R b/R/object-package.R
@@ -51,7 +51,7 @@ author_desc <- function(x) {
   }
 
   if (!is.null(x$email)) {
-    desc <- paste0(desc, " \\email{", x$email, "}")
+    desc <- paste0(desc, " \\email{", paste(x$email, collapse = ", "), "}")
   }
 
   if (!is.null(x$comment)) {

diff --git a/tests/testthat/test-object-package.R b/tests/testthat/test-object-package.R
@@ -75,3 +75,11 @@ test_that("autolink several matching patterns", {
     )
   )
 })
+
+test_that("multiple email addresses for a person are acceptable #1487", {
+  me <- person("me", email = c("one@email.me", "two@email.me"))
+  expect_equal(
+    author_desc(unclass(me)[[1]]),
+    "me \\email{one@email.me, two@email.me}"
+  )
+})