Skip to content

Commit

Permalink
Merge pull request #183 from astamm/ci-nlopt-multi-versions
Browse files Browse the repository at this point in the history
Tentative GHA for CI against all versions of nlopt since 2.7.0.
  • Loading branch information
astamm authored Mar 10, 2025
2 parents 5a3303b + 49a5091 commit 3884bb1
Show file tree
Hide file tree
Showing 3 changed files with 84 additions and 8 deletions.
67 changes: 67 additions & 0 deletions .github/workflows/R-CMD-check-system.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,67 @@
# 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
on:
push:
branches: [main, master, devel]
pull_request:
branches: [main, master, devel]

name: R-CMD-check-system.yaml

permissions: read-all

jobs:
R-CMD-check:
runs-on: ${{ matrix.config.os }}

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

strategy:
fail-fast: false
matrix:
config:
- {os: ubuntu-latest, r: 'release', nlopt: '2.7.0' }
- {os: ubuntu-latest, r: 'release', nlopt: '2.7.1' }
- {os: ubuntu-latest, r: 'release', nlopt: '2.8.0' }
- {os: ubuntu-latest, r: 'release', nlopt: '2.9.0' }
- {os: ubuntu-latest, r: 'release', nlopt: '2.9.1' }
- {os: ubuntu-latest, r: 'release', nlopt: '2.10.0'}

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 }}
use-public-rspm: true

- uses: jwlawson/[email protected]
with:
cmake-version: '3.22.x'

- name: Install nlopt
run: |
curl -OLs https://github.com/stevengj/nlopt/archive/refs/tags/v${{ matrix.config.nlopt }}.tar.gz
chmod 0755 v${{ matrix.config.nlopt }}.tar.gz
tar xzvf v${{ matrix.config.nlopt }}.tar.gz
chmod 0755 nlopt-${{ matrix.config.nlopt }}
cd nlopt-${{ matrix.config.nlopt }}
cmake . && make && sudo make install
cd ..
rm -fr nlopt-${{ matrix.config.nlopt }} v${{ matrix.config.nlopt }}.tar.gz
- uses: r-lib/actions/setup-r-dependencies@v2
with:
extra-packages: any::rcmdcheck
needs: check

- uses: r-lib/actions/check-r-package@v2
with:
upload-snapshots: true
build_args: 'c("--no-manual","--compact-vignettes=gs+qpdf")'
10 changes: 5 additions & 5 deletions .github/workflows/R-CMD-check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,11 +20,11 @@ jobs:
fail-fast: false
matrix:
config:
- {os: macos-latest, r: 'release'}
- {os: windows-latest, r: 'release'}
- {os: ubuntu-latest, r: 'devel', http-user-agent: 'release'}
- {os: ubuntu-latest, r: 'release'}
- {os: ubuntu-latest, r: 'oldrel-1'}
- {os: macos-latest, r: 'release' }
- {os: windows-latest, r: 'release' }
- {os: ubuntu-latest, r: 'devel', http-user-agent: 'release'}
- {os: ubuntu-latest, r: 'release' }
- {os: ubuntu-latest, r: 'oldrel-1' }

env:
GITHUB_PAT: ${{ secrets.GITHUB_TOKEN }}
Expand Down
15 changes: 12 additions & 3 deletions cran-comments.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,11 @@
* macOS latest release
* windows latest release
* windows latest release with Rtools42
* ubuntu 22.04 latest release and devel using the *building nlopt from source*
* Ubuntu 24.04.2 LTS latest release and devel using the *building nlopt from source*
strategy
* ubuntu 22.04 latest release using the *reuse existing build of nlopt*
* Ubuntu 24.04.2 LTS latest release using the *reuse existing build of nlopt*
strategy
* Ubuntu 24.04.2 LTS against all versions of nlopt since `2.7.0`.
* [win-builder](https://win-builder.r-project.org/) (release and devel) NOTHING TO DO
* [macOS-builder](https://mac.r-project.org/macbuilder/submit.html) CHECKER BROKEN
* [R-hub](https://builder.r-hub.io)
Expand All @@ -25,7 +26,15 @@ There was no ERROR, no WARNING and no NOTE.

We checked 160 reverse dependencies (151 from CRAN + 9 from Bioconductor), comparing R CMD check results across CRAN and dev versions of this package.

* We saw 0 new problems,
* We saw 1 new problem (**smooth**):

- we filed an issue [here](https://github.com/astamm/nloptr/issues/182) to
keep track of it,
- we investigated the issue and found it comes from **smooth**,
- we informed the author of **smooth** by filing an issue in his repo
[here](https://github.com/config-i1/smooth/issues/242),
- he replied back and we are working on a solution.

* We failed to check 3 packages

Issues with CRAN packages are summarised below.
Expand Down

0 comments on commit 3884bb1

Please sign in to comment.