Skip to content

Commit ec3e68e

Browse files
committed
implement this in a new yaml file instead
1 parent 2ca5d01 commit ec3e68e

3 files changed

Lines changed: 37 additions & 5 deletions

File tree

.github/workflows/ci.yaml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,6 @@ jobs:
5252
- name: dev
5353
cntr: rcpp/ci-dev
5454
r: RD
55-
cxxflags: -Werror
5655

5756
steps:
5857
- uses: actions/checkout@v6
@@ -63,9 +62,6 @@ jobs:
6362
- name: Build
6463
run: ${{matrix.r }} CMD build --no-build-vignettes --no-manual .
6564

66-
- name: Set flags
67-
run: mkdir -p ~/.R && echo "CXXFLAGS += ${{ matrix.cxxflags }}" >> ~/.R/Makevars
68-
6965
- name: Check
7066
run: CI=true ${{ matrix.r }} CMD check --no-vignettes --no-manual Rcpp_*.tar.gz
7167

.github/workflows/warnings.yaml

Lines changed: 36 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,36 @@
1+
# Run CI for R using https://eddelbuettel.github.io/r-ci/
2+
3+
name: warnings
4+
5+
on:
6+
push:
7+
pull_request:
8+
9+
env:
10+
_R_CHECK_FORCE_SUGGESTS_: "false"
11+
RCPP_CXXFLAGS: "-Werror"
12+
13+
jobs:
14+
ci:
15+
runs-on: ubuntu-latest
16+
container: rocker/drd
17+
18+
steps:
19+
- name: Checkout
20+
uses: actions/checkout@v7
21+
22+
- name: Setup
23+
uses: eddelbuettel/github-actions/r-ci@master
24+
25+
- name: Dependencies
26+
run: ./run.sh install_deps
27+
28+
- name: Set flags
29+
run: mkdir -p ~/.R && echo "CXXFLAGS += $RCPP_CXXFLAGS" >> ~/.R/Makevars
30+
31+
- name: Test
32+
run: ./run.sh run_tests
33+
34+
- name: Logs
35+
run: ./run.sh dump_logs
36+
if: failure()

ChangeLog

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
2026-09-01 Iñaki Ucar <iucar@fedoraproject.org>
22

3-
* .github/workflows/ci.yaml: Add -Werror to CXXFLAGS for dev CI job
3+
* .github/workflows/warnings.yaml: Add new CI file with -Werror enabled
44

55
2026-08-08 Dirk Eddelbuettel <edd@debian.org>
66

0 commit comments

Comments
 (0)