Skip to content

Commit

Permalink
Disable CI on macOS (#19)
Browse files Browse the repository at this point in the history
* Another attempt to fix macos CI

* Need basilisk as well

* Is this problem Arm specific?

* Try setting the `BASILISK_USE_SYSTEM_DIR` envvar

* Only run actions on most recent push

* Set `BASILISK_USE_SYSTEM_DIR` envvar for all steps

* Install basilisk from source

* Does this work?

See conda/conda#7113

* Install basilisk after other R depencencies

* Only do this on macOS

* Update roxygen version

* Fix typo

* Build ignore more files

* Manually trigger basilisk env configuration

* Don't use system dir for basilisk?

* Invalidate dependencies cache and move basilisk installation up a step

* Revert non-working fixes

* Disable macOS workflow

See #28
  • Loading branch information
milanmlft authored May 10, 2024
1 parent 7bec6fe commit 38e0669
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 6 deletions.
3 changes: 3 additions & 0 deletions .Rbuildignore
Original file line number Diff line number Diff line change
Expand Up @@ -6,3 +6,6 @@
^_pkgdown\.yml$
^docs$
^pkgdown$
^\.lintr$
^\.renovaterc\.json5$
^\.vscode$
14 changes: 9 additions & 5 deletions .github/workflows/R-CMD-check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,11 @@ on:

name: R-CMD-check

# Only run actions on the most recent push to a branch
concurrency:
group: "${{ github.workflow }}-${{ github.head_ref }}"
cancel-in-progress: true

jobs:
R-CMD-check:
runs-on: ${{ matrix.config.os }}
Expand All @@ -19,7 +24,9 @@ jobs:
fail-fast: false
matrix:
config:
- { os: macos-latest, r: "release" }
# TODO: fix broken macOS workflow
# https://github.com/milanmlft/jupytextR/issues/28
# - { os: macos-latest, r: "release" }
- { os: windows-latest, r: "release" }
- { os: ubuntu-latest, r: "devel", http-user-agent: "release" }
- { os: ubuntu-latest, r: "release" }
Expand All @@ -43,13 +50,10 @@ jobs:
http-user-agent: ${{ matrix.config.http-user-agent }}
use-public-rspm: true

# Install miniconda manually, to prevent installation errors on GHA runners
- name: Setup Miniconda
uses: conda-incubator/setup-miniconda@a4260408e20b96e80095f42ff7f1a15b27dd94ca # v3.0.4

- name: Install R dependencies
uses: r-lib/actions/setup-r-dependencies@929c772977a3a13c8733b363bf5a2f685c25dd91 # v2
with:
cache-version: 3
extra-packages: any::rcmdcheck
needs: check

Expand Down
2 changes: 1 addition & 1 deletion DESCRIPTION
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ Suggests:
testthat (>= 3.0.0)
Config/testthat/edition: 3
Roxygen: list(markdown = TRUE)
RoxygenNote: 7.2.3
RoxygenNote: 7.3.1
URL: https://github.com/milanmlft/jupytextR, https://milanmlft.github.io/jupytextR/
BugReports: https://github.com/milanmlft/jupytextR/issues
StagedInstall: no

0 comments on commit 38e0669

Please sign in to comment.