Merge branch 'main' into f-558-prof

Author: krlmlr

.github/workflows/check-full.yaml

@@ -27,12 +27,12 @@ jobs:
           - {os: windows-latest, r: '3.6'}
 
           # Use older ubuntu to maximise backward compatibility
-          - {os: ubuntu-18.04,   r: 'devel', http-user-agent: 'release'}
-          - {os: ubuntu-18.04,   r: 'release'}
-          - {os: ubuntu-18.04,   r: 'oldrel-1'}
-          - {os: ubuntu-18.04,   r: 'oldrel-2'}
-          - {os: ubuntu-18.04,   r: 'oldrel-3'}
-          - {os: ubuntu-18.04,   r: 'oldrel-4'}
+          - {os: ubuntu-latest,  r: 'devel', http-user-agent: 'release'}
+          - {os: ubuntu-latest,  r: 'release'}
+          - {os: ubuntu-latest,  r: 'oldrel-1'}
+          - {os: ubuntu-latest,  r: 'oldrel-2'}
+          - {os: ubuntu-latest,  r: 'oldrel-3'}
+          - {os: ubuntu-latest,  r: 'oldrel-4'}
 
     env:
       GITHUB_PAT: ${{ secrets.GITHUB_TOKEN }}
@@ -51,10 +51,12 @@ jobs:
 
       - uses: r-lib/actions/setup-r-dependencies@v2
         with:
-          extra-packages: any::rcmdcheck, data.tree=?ignore-before-r=3.5.0
+          extra-packages: any::rcmdcheck
           needs: check
 
       - uses: r-lib/actions/check-r-package@v2
+        with:
+          error-on: '"note"'
         env:
           _R_CHECK_FORCE_SUGGESTS_: false
 

.github/workflows/pre-commit.yaml

@@ -9,7 +9,7 @@ on:
 
 jobs:
   pre-commit:
-    runs-on: ubuntu-18.04
+    runs-on: ubuntu-latest
     if: >-
       !contains(github.event.head_commit.message, 'ci skip') &&
       (

DESCRIPTION

@@ -11,14 +11,19 @@ Authors@R:
       person(given = "Lorenz",
              family = "Walthert",
              role = c("cre", "aut"),
-             email = "[email protected]"))
+             email = "[email protected]"),
+      person(given = "Indrajeet",
+             family = "Patil",
+             role = "ctb",
+             email = "[email protected]",
+             comment = c(ORCID = "0000-0003-1995-6531", Twitter = "@patilindrajeets")))
 Description: Pretty-prints R code without changing the user's formatting
     intent.
 License: MIT + file LICENSE
 URL: https://github.com/r-lib/styler, https://styler.r-lib.org
 BugReports: https://github.com/r-lib/styler/issues
 Depends:
-    R (>= 3.4.0)
+    R (>= 3.5.0)
 Imports: 
     cli (>= 3.1.1),
     magrittr (>= 2.0.0),
@@ -48,6 +53,7 @@ Roxygen: list(markdown = TRUE, roclets = c("rd", "namespace", "collate",
     "pkgapi::api_roclet"))
 RoxygenNote: 7.2.1
 Config/testthat/edition: 3
+Config/testthat/parallel: true
 Collate: 
     'addins.R'
     'communicate.R'

NEWS.md

@@ -11,7 +11,8 @@ editor_options:
 -  `style_dir()` and `style_pkg()` now default to styling all supported file 
    formats (`.R`, `.Rmd`, `.Rmarkdown`, `.Rnw`, and `.qmd`) in the (package) 
    directory (\@IndrajeetPatil, #965).
-- `style_pkg()` now excludes the auto-generated `R/cpp11.R` file (#977).
+-  `style_pkg()` now excludes the auto-generated `R/cpp11.R` file (#977).
+-   Minimum needed R version is now bumped to `3.5` (\@IndrajeetPatil, #986).
 
 **Features**
 
@@ -49,6 +50,10 @@ editor_options:
 
 **Other**
 
+-   \@IndrajeetPatil is now a contributor to {styler}. Welcome and thanks for
+    everything you did so far! (#988).
+-   Non-exported and unused functions `odd()` and `even()` were removed 
+    (\@IndrajeetPatil, #989).
 -   Old (and outdated) vignettes have been removed (\@IndrajeetPatil, #955). To
     access them, do `git checkout v1.0.0`.
 -   Upgrade testing infra to testthat 3e (\@IndrajeetPatil, #949).
@@ -628,7 +633,7 @@ Thanks to all contributors involved, in particular
 ## Adaption
 
 -   styler is now available through the pre-commit hook `style-files` in
-    <https://github.com/lorenzwalthert/pre-commit-hooks>.
+    <https://github.com/lorenzwalthert/precommit>.
 
 Thanks to all contributors involved, in particular
 

R/expr-is.R

@@ -103,7 +103,7 @@ is_subset_expr <- function(pd) {
 #' Identify comments that are shebangs
 #'
 #' Shebangs should be preserved and no space should be inserted between
-#' \# and !. A comment is a shebang if it is the first top level token
+#' `#` and `!`. A comment is a shebang if it is the first top level token
 #' (identified with `pos_id`) and if it starts with `#!`.
 #' @param pd A parse table.
 #' @examples

R/utils.R

@@ -48,24 +48,13 @@ two_cols_match <- function(col1, col2, data) {
   all(unlist(data[col1]) == unlist(data[col2]))
 }
 
-odd <- function(x) {
-  x[odd_index(x)]
-}
-
 odd_index <- function(x) {
   if (length(x) < 1) {
     return(NULL)
   }
   seq(1L, length(x), by = 2)
 }
 
-even <- function(x) {
-  if (length(x) < 2) {
-    return(NULL)
-  }
-  x[even_index(x)]
-}
-
 even_index <- function(x) {
   seq(2L, length(x), by = 2)
 }

cran-comments.md

@@ -10,7 +10,7 @@ Luke Tierney.
 
 ## Test environments
 
--   ubuntu 18.04 (on GitHub Actions): R devel, R 4.1.2, R 4.0.5, R 3.6, R 3.5,
+-   ubuntu 20.04 (on GitHub Actions): R devel, R 4.1.2, R 4.0.5, R 3.6, R 3.5,
     R 3.4
 -   Windows Server 10 (on GitHub Actions): R 3.6, R 4.0.5
 -   win-builder: R devel

inst/WORDLIST

@@ -31,6 +31,7 @@ covr
 cpp
 cran
 cre
+ctb
 cynkra
 datastructures
 dec
@@ -75,6 +76,7 @@ ggplot
 ghclass
 github
 gitsum
+gmail
 grkstyle
 GSOC
 hashFiles
@@ -85,6 +87,7 @@ https
 icloud
 ifelse
 impl
+Indrajeet
 infinitively
 initializer
 inode
@@ -141,6 +144,9 @@ params
 paren
 parsable
 parsesum
+Patil
+patilindrajeet
+patilindrajeets
 pgkdown
 pkgapi
 pkgdown