Skip to content

Commit 803dfc1

Browse files
authored
ci: fail on note (#350)
* ci: fail on note * ...
1 parent fcfb057 commit 803dfc1

File tree

4 files changed

+93
-5
lines changed

4 files changed

+93
-5
lines changed

.github/workflows/dev-cmd-check.yml

Lines changed: 17 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,13 @@
1-
# dev cmd check workflow of the mlr3 ecosystem v0.2.0
1+
# dev cmd check workflow of the mlr3 ecosystem v0.4.0
22
# https://github.com/mlr-org/actions
33
on:
44
workflow_dispatch:
5+
inputs:
6+
debug_enabled:
7+
type: boolean
8+
description: 'Run the build with tmate debugging enabled'
9+
required: false
10+
default: false
511
push:
612
branches:
713
- main
@@ -25,10 +31,11 @@ jobs:
2531
matrix:
2632
config:
2733
- {os: ubuntu-latest, r: 'release', dev-package: 'mlr-org/mlr3'}
34+
- {os: ubuntu-latest, r: 'release', dev-package: 'mlr-org/mlr3misc'}
2835
- {os: ubuntu-latest, r: 'release', dev-package: 'mlr-org/paradox'}
2936

3037
steps:
31-
- uses: actions/checkout@v4
38+
- uses: actions/checkout@v3
3239

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

@@ -40,9 +47,17 @@ jobs:
4047
with:
4148
extra-packages: any::rcmdcheck
4249
needs: check
50+
error-on: '"note"'
4351

4452
- name: Install dev versions
4553
run: pak::pkg_install('${{ matrix.config.dev-package }}')
4654
shell: Rscript {0}
4755

56+
- uses: mxschmitt/action-tmate@v3
57+
if: ${{ github.event_name == 'workflow_dispatch' && inputs.debug_enabled }}
58+
with:
59+
limit-access-to-actor: true
60+
4861
- uses: r-lib/actions/check-r-package@v2
62+
with:
63+
args: 'c("--no-manual", "--as-cran")'
Lines changed: 58 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,58 @@
1+
# r cmd check workflow without suggests of the mlr3 ecosystem v0.1.0
2+
# https://github.com/mlr-org/actions
3+
on:
4+
workflow_dispatch:
5+
inputs:
6+
debug_enabled:
7+
type: boolean
8+
description: 'Run the build with tmate debugging enabled'
9+
required: false
10+
default: false
11+
push:
12+
branches:
13+
- main
14+
pull_request:
15+
branches:
16+
- main
17+
18+
name: no-suggest-cmd-check
19+
20+
jobs:
21+
no-suggest-cmd-check:
22+
runs-on: ${{ matrix.config.os }}
23+
24+
name: ${{ matrix.config.os }} (${{ matrix.config.r }})
25+
26+
env:
27+
GITHUB_PAT: ${{ secrets.GITHUB_TOKEN }}
28+
29+
strategy:
30+
fail-fast: false
31+
matrix:
32+
config:
33+
- {os: ubuntu-latest, r: 'release'}
34+
35+
steps:
36+
- uses: actions/checkout@v3
37+
38+
- uses: r-lib/actions/setup-pandoc@v2
39+
40+
- uses: r-lib/actions/setup-r@v2
41+
with:
42+
r-version: ${{ matrix.config.r }}
43+
44+
- uses: r-lib/actions/setup-r-dependencies@v2
45+
with:
46+
extra-packages: any::rcmdcheck
47+
needs: check
48+
error-on: '"note"'
49+
dependencies: '"hard"'
50+
51+
- uses: mxschmitt/action-tmate@v3
52+
if: ${{ github.event_name == 'workflow_dispatch' && inputs.debug_enabled }}
53+
with:
54+
limit-access-to-actor: true
55+
56+
- uses: r-lib/actions/check-r-package@v2
57+
with:
58+
args: 'c("--no-manual", "--as-cran")'

.github/workflows/r-cmd-check.yml

Lines changed: 16 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,13 @@
1-
# r cmd check workflow of the mlr3 ecosystem v0.2.0
1+
# r cmd check workflow of the mlr3 ecosystem v0.4.0
22
# https://github.com/mlr-org/actions
33
on:
44
workflow_dispatch:
5+
inputs:
6+
debug_enabled:
7+
type: boolean
8+
description: 'Run the build with tmate debugging enabled'
9+
required: false
10+
default: false
511
push:
612
branches:
713
- main
@@ -28,7 +34,7 @@ jobs:
2834
- {os: ubuntu-latest, r: 'release'}
2935

3036
steps:
31-
- uses: actions/checkout@v4
37+
- uses: actions/checkout@v3
3238

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

@@ -40,5 +46,13 @@ jobs:
4046
with:
4147
extra-packages: any::rcmdcheck
4248
needs: check
49+
error-on: '"note"'
50+
51+
- uses: mxschmitt/action-tmate@v3
52+
if: ${{ github.event_name == 'workflow_dispatch' && inputs.debug_enabled }}
53+
with:
54+
limit-access-to-actor: true
4355

4456
- uses: r-lib/actions/check-r-package@v2
57+
with:
58+
args: 'c("--no-manual", "--as-cran")'

R/LearnerRegrKM.R

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,8 @@ LearnerRegrKM = R6Class("LearnerRegrKM",
116116
newdata = newdata,
117117
type = if (is.null(pv$type)) "SK" else pv$type,
118118
se.compute = self$predict_type == "se",
119-
.args = remove_named(pv, "jitter")
119+
.args = remove_named(pv, "jitter"),
120+
.opts = list(warnPartialMatchArgs = FALSE)
120121
)
121122

122123
list(response = p$mean, se = p$sd)

0 commit comments

Comments
 (0)