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

Improve build CI #113

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open
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
5 changes: 2 additions & 3 deletions .github/workflows/pre-commit.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# https://pre-commit.com
# This GitHub Action assumes that the repo contains a valid .pre-commit-config.yaml file.
name: pre-commit
name: Run Pre-commit
on:
pull_request:
push:
Expand All @@ -16,8 +16,7 @@ jobs:
- uses: actions/checkout@v3
- uses: actions/setup-python@v5
with:
python-version: '3.9'
python-version: '3.12'
- run: python -m pip install pre-commit
- run: python -m pre_commit --version
- run: python -m pre_commit install
- run: python -m pre_commit run --all-files
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,9 @@ jobs:
name: Build wheels on ${{ matrix.os }}
runs-on: ${{ matrix.os }}
strategy:
fail-fast: false
matrix:
os: [ubuntu-22.04] #, macos-11] - disable macos, as it's not working for a moment
os: [ubuntu-latest, macos-12]

steps:
- uses: actions/checkout@v3
Expand All @@ -36,8 +37,8 @@ jobs:
uses: pypa/[email protected]
env:
# Configure cibuildwheel to build native archs, and some emulated ones
CIBW_ARCHS_LINUX: x86_64 # aarch64 - disable ARM, as it's not working for a moment
#CIBW_ARCHS_MACOS: x86_64 arm64 - macos is disabled for a moment
CIBW_ARCHS_LINUX: x86_64 aarch64
CIBW_ARCHS_MACOS: x86_64 arm64
CIBW_BUILD_VERBOSITY: 3 # Increase verbosity to see what's going on
CIBW_REPAIR_WHEEL_COMMAND_LINUX: > # Print additional info from auditwheel
auditwheel show {wheel} && auditwheel repair -w {dest_dir} {wheel}
Expand Down
16 changes: 12 additions & 4 deletions .github/workflows/test-cpp.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: Test C++ Stable-Retro code
name: Run CTest

on: [pull_request, push]

Expand All @@ -7,14 +7,22 @@ permissions:

jobs:
build:
runs-on: ubuntu-latest # todo, add more OS systems to see if they work as well

name: ${{ matrix.runs_on }}
strategy:
fail-fast: false
matrix:
include:
- runs-on: ubuntu-latest
- runs-on: mac-12
- runs-on: mac-14

runs-on: ${{ matrix.runs-on }}
steps:
- uses: actions/checkout@v3

- uses: actions/setup-python@v5
with:
python-version: 3.9
python-version: 3.12

- name: Install system packages
run: |
Expand Down
16 changes: 12 additions & 4 deletions .github/workflows/test-python.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: Test Stable-Retro
name: Run PyTest

on: [pull_request, push]

Expand All @@ -7,11 +7,19 @@ permissions:

jobs:
build:
runs-on: ubuntu-latest # todo, add more OS systems to see if they work as well
strategy:
fail-fast: false
matrix:
python-version: ['3.8', '3.9', '3.10', '3.11', '3.12']

include:
- runs-on: ubuntu-latest
python: '3.8'
- runs-on: mac-12 # x86_64
python: '3.8'
- runs-on: mac-14 # arm64
python: '3.8'

runs-on: ${{ matrix.runs-on }}
name: ${{ matrix.runs-on }} • py${{ matrix.python }}
steps:
- uses: actions/checkout@v3

Expand Down
Loading