Skip to content
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

updates for no-suggests checks #70

Merged
merged 5 commits into from
Oct 16, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
59 changes: 59 additions & 0 deletions .github/workflows/R-CMD-check-hard.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,59 @@
# Workflow derived from https://github.com/r-lib/actions/tree/v2/examples
# Need help debugging build failures? Start at https://github.com/r-lib/actions#where-to-find-help
#
# NOTE: This workflow only directly installs "hard" dependencies, i.e. Depends,
# Imports, and LinkingTo dependencies. Notably, Suggests dependencies are never
# installed, with the exception of testthat, knitr, and rmarkdown. The cache is
# never used to avoid accidentally restoring a cache containing a suggested
# dependency.
on:
push:
branches: [main, master]
pull_request:

name: R-CMD-check-hard.yaml

permissions: read-all

jobs:
check-no-suggests:
runs-on: ${{ matrix.config.os }}

name: ${{ matrix.config.os }} (${{ matrix.config.r }})

strategy:
fail-fast: false
matrix:
config:
- {os: ubuntu-latest, r: 'release'}

env:
GITHUB_PAT: ${{ secrets.GITHUB_TOKEN }}
R_KEEP_PKG_SOURCE: yes

steps:
- uses: actions/checkout@v4

- uses: r-lib/actions/setup-pandoc@v2

- uses: r-lib/actions/setup-r@v2
with:
r-version: ${{ matrix.config.r }}
http-user-agent: ${{ matrix.config.http-user-agent }}
use-public-rspm: true

- uses: r-lib/actions/setup-r-dependencies@v2
with:
dependencies: '"hard"'
cache: false
extra-packages: |
any::rcmdcheck
any::testthat
any::knitr
any::rmarkdown
needs: check

- uses: r-lib/actions/check-r-package@v2
with:
upload-snapshots: true
build_args: 'c("--no-manual","--compact-vignettes=gs+qpdf")'
11 changes: 5 additions & 6 deletions DESCRIPTION
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,9 @@ Version: 1.0.1.9000
Authors@R: c(
person("Emil", "Hvitfeldt", , "[email protected]", role = c("aut", "cre"),
comment = c(ORCID = "0000-0002-0679-1945")),
person("Max", "Kuhn", , "[email protected]", role = c("aut"),
person("Max", "Kuhn", , "[email protected]", role = "aut",
comment = c(ORCID = "0000-0003-2402-136X")),
person(given = "Posit Software, PBC", role = c("cph", "fnd"))
person("Posit Software, PBC", role = c("cph", "fnd"))
)
Description: Bindings for additional classification models for use with
the 'parsnip' package. Models include flavors of discriminant
Expand Down Expand Up @@ -34,6 +34,7 @@ Suggests:
dplyr,
earth,
ggplot2,
hardhat,
klaR,
knitr,
MASS,
Expand All @@ -47,12 +48,10 @@ Suggests:
spelling,
testthat (>= 3.0.0),
xml2
Config/Needs/website:
tidymodels/tidymodels,
tidyverse/tidytemplate
Config/Needs/website: tidymodels/tidymodels, tidyverse/tidytemplate
Config/testthat/edition: 3
Encoding: UTF-8
Language: en-US
LazyData: true
Roxygen: list(markdown = TRUE)
RoxygenNote: 7.3.2
Config/testthat/edition: 3
12 changes: 7 additions & 5 deletions R/data.R
Original file line number Diff line number Diff line change
Expand Up @@ -10,11 +10,13 @@
#'
#' @keywords datasets
#' @examples
#' data(parabolic)
#' if (rlang::is_installed("ggplot2")) {
#' data(parabolic)
#'
#' library(ggplot2)
#' ggplot(parabolic, aes(x = X1, y = X2, col = class)) +
#' geom_point(alpha = .5) +
#' theme_bw()
#' library(ggplot2)
#' ggplot(parabolic, aes(x = X1, y = X2, col = class)) +
#' geom_point(alpha = .5) +
#' theme_bw()
#' }
#'
NULL
47 changes: 15 additions & 32 deletions inst/WORDLIST
Original file line number Diff line number Diff line change
@@ -1,43 +1,26 @@
al
Ahdesmaki
Buja
CMD
Codecov
doi
et
Hastie
klaR
Lifecycle
mda
nonlinearly
param
pre
ORCID
PBC
QDA
quant
RStudio
Strimmer
Tibshirani
Yu
cov
fL
frac
laplace
doi
funder
klaR
mda
modeldata
naivebayes
nprune
num
pmethod
param
psock
Ahdesmaki
Strimmer
quant
reprex
sda
sparsediscrim
Bioinformatics
Biometrics
Dudoit
Fridlyand
Kubokawa
Springer
Srivistava
Zhao
backward'
diagonal'
naivebayes
ORCID
modeldata
tidymodels
funder
12 changes: 7 additions & 5 deletions man/parabolic.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

30 changes: 0 additions & 30 deletions tests/testthat/_snaps/flexible.md

This file was deleted.

4 changes: 2 additions & 2 deletions tests/testthat/_snaps/linear-lda.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# missing data
# MASS::lda missing data

Code
f_pred <- predict(f_fit, glass_na, type = "prob")
Expand All @@ -11,7 +11,7 @@
---

Code
exp_f_pred <- probs_to_tibble(predict(exp_f_fit, glass_na)$posterior)
exp_f_pred <- probs_to_tibble(predict(exp_f_fit_lda, glass_na)$posterior)
Condition
Warning in `FUN()`:
no non-missing arguments to min; returning Inf
Expand Down
2 changes: 1 addition & 1 deletion tests/testthat/_snaps/linear-sparsediscrim.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# lda_emp_bayes_eigen fit and prediction
# sparsediscrim lda_emp_bayes_eigen fit and prediction

'method' should be one of: 'diagonal', 'min_distance', 'shrink_cov', 'shrink_mean'

13 changes: 0 additions & 13 deletions tests/testthat/_snaps/naive-Bayes.md

This file was deleted.

13 changes: 0 additions & 13 deletions tests/testthat/_snaps/naive-Bayes_naivebayes.md

This file was deleted.

28 changes: 2 additions & 26 deletions tests/testthat/_snaps/quad-qda.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# missing data
# MASS::qda missing data

Code
f_pred <- predict(f_fit, penguins_miss, type = "prob")
Expand Down Expand Up @@ -28,29 +28,5 @@

---

Code
exp_f_pred <- probs_to_tibble(predict(exp_f_fit, penguins_miss)$posterior)
Condition
Warning in `FUN()`:
no non-missing arguments to min; returning Inf
Warning in `FUN()`:
no non-missing arguments to min; returning Inf
Warning in `FUN()`:
no non-missing arguments to min; returning Inf
Warning in `FUN()`:
no non-missing arguments to min; returning Inf
Warning in `FUN()`:
no non-missing arguments to min; returning Inf
Warning in `FUN()`:
no non-missing arguments to min; returning Inf
Warning in `FUN()`:
no non-missing arguments to min; returning Inf
Warning in `FUN()`:
no non-missing arguments to min; returning Inf
Warning in `FUN()`:
no non-missing arguments to min; returning Inf
Warning in `FUN()`:
no non-missing arguments to min; returning Inf
Warning in `FUN()`:
no non-missing arguments to min; returning Inf
no non-missing arguments to min; returning Inf

2 changes: 1 addition & 1 deletion tests/testthat/_snaps/rda.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# printing
# klaR::rda printing

Code
print(rda_spec)
Expand Down
Loading
Loading