Skip to content

actions: use mamba to provision Python environment #363

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.

Sign up for GitHub

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

Merged
MetRonnie merged 2 commits into from
Mar 11, 2025
Merged

actions: use mamba to provision Python environment #363

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
bd2a1df
actions: use mamba to provision Python environment
oliver-sanders Feb 10, 2025
3aa6c56
actions: add "pipefail" protection to tests
oliver-sanders Mar 11, 2025
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
22 changes: 17 additions & 5 deletions .github/workflows/tests.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,13 +15,17 @@ concurrency:
env:
FORCE_COLOR: 2

defaults:
run:
shell: bash -elo pipefail {0}
Copy link
Copy Markdown
Member

@MetRonnie MetRonnie Mar 11, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be (like in cylc/cylc-flow#6655)?

Suggested change
shell: bash -elo pipefail {0}
shell: bash -c "exec $CONDA_PREFIX/bin/bash -elo pipefail {0}"

Copy link
Copy Markdown
Member Author

@oliver-sanders oliver-sanders Mar 11, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

$CONDA_PREFIX/bin/bash points at the bash executable installed by Micromamba.

These Python tests don't have such shell requirements so this PR sticks with using the system installed Bash.


jobs:
test:
runs-on: ubuntu-latest
timeout-minutes: 5
strategy:
matrix:
python-version: ['3.7', '3.8', '3.9', '3.10', '3.11']
python-version: ['3.7', '3.8', '3.9', '3.10', '3.11', '3.12', '3']
fail-fast: false
env:
PYTEST_ADDOPTS: --cov --cov-append --color=yes
Expand All @@ -30,9 +34,13 @@ jobs:
uses: actions/checkout@v4

- name: Configure Python
uses: actions/setup-python@v5
uses: mamba-org/setup-micromamba@v2
with:
python-version: ${{ matrix.python-version }}
cache-environment: true
post-cleanup: 'all'
environment-name: cylc
create-args: >-
python=${{ matrix.python-version }}

- name: install libs
uses: cylc/release-actions/install-cylc-components@v1
Expand Down Expand Up @@ -111,9 +119,13 @@ jobs:
uses: actions/checkout@v4

- name: Configure Python
uses: actions/setup-python@v5
uses: mamba-org/setup-micromamba@v2
with:
python-version: '3.9'
cache-environment: true
post-cleanup: 'all'
environment-name: cylc
create-args: >-
python=${{ matrix.python-version }}

- name: install libs
uses: cylc/release-actions/install-cylc-components@v1
Expand Down