Skip to content

Commit

Permalink
WIP: Switched the backend to virtualenv. (#299)
Browse files Browse the repository at this point in the history
* Switched the backend to virtualenv. Fixed #298
* Temporary mod for #302

---------

Co-authored-by: Sylvain MARIE <sylvain.marie@se.com>
  • Loading branch information
smarie and Sylvain MARIE committed Oct 4, 2023
1 parent ab3b719 commit 8c2e707
Show file tree
Hide file tree
Showing 8 changed files with 397 additions and 247 deletions.
107 changes: 63 additions & 44 deletions .github/workflows/base.yml
Expand Up @@ -12,24 +12,32 @@ on:
pull_request:
branches:
- main

defaults:
run:
shell: bash -l {0}

jobs:
# pre-job to read nox tests matrix - see https://stackoverflow.com/q/66747359/7262247
list_nox_test_sessions:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- uses: actions/setup-python@v1
- name: Checkout
uses: actions/checkout@v2

- name: Install python 3.9
uses: actions/setup-python@v4
with:
python-version: 3.7
python-version: 3.9
architecture: x64

- name: Install noxfile requirements
shell: bash -l {0}
run: pip install -r noxfile-requirements.txt

- name: List 'tests' nox sessions
- name: List 'tests' nox sessions and required python versions
id: set-matrix
run: echo "::set-output name=matrix::$(nox -s gha_list -- tests)"
run: echo "::set-output name=matrix::$(nox -s gha_list -- -s tests -v)"

outputs:
matrix: ${{ steps.set-matrix.outputs.matrix }} # save nox sessions list to outputs

Expand All @@ -38,36 +46,39 @@ jobs:
strategy:
fail-fast: false
matrix:
# see https://github.com/actions/setup-python/issues/544
# os: [ ubuntu-20.04 ]
os: [ ubuntu-latest ] # , macos-latest, windows-latest]
# all nox sessions: manually > dynamically from previous job
# nox_session: ["tests-2.7(env='pytest2.x')", "tests-3.7(env='pytest-latest')"]
nox_session: ${{ fromJson(needs.list_nox_test_sessions.outputs.matrix) }}

name: ${{ matrix.os }} ${{ matrix.nox_session }} # ${{ matrix.name_suffix }}
name: ${{ matrix.os }} ${{ matrix.nox_session.python }} ${{ matrix.nox_session.session }} # ${{ matrix.name_suffix }}
runs-on: ${{ matrix.os }}
steps:
- uses: actions/checkout@v2
- name: Checkout
uses: actions/checkout@v2

- name: Install python ${{ matrix.nox_session.python }} for tests
uses: MatteoH2O1999/setup-python@v1 # actions/setup-python@v4
id: set-py
with:
python-version: ${{ matrix.nox_session.python }}
architecture: x64
allow-build: info
cache-build: true

# Conda install
- name: Install conda v3.7
uses: conda-incubator/setup-miniconda@v2
- name: Install python 3.9 for nox
uses: actions/setup-python@v4
with:
# auto-update-conda: true
python-version: 3.7
activate-environment: noxenv
- run: conda info
shell: bash -l {0} # so that conda works
- run: conda list
shell: bash -l {0} # so that conda works

# Nox install + run
python-version: 3.9
architecture: x64

- name: Install noxfile requirements
shell: bash -l {0} # so that conda works
run: pip install -r noxfile-requirements.txt
- run: conda list
shell: bash -l {0} # so that conda works
- run: nox -s "${{ matrix.nox_session }}"
shell: bash -l {0} # so that conda works

- name: Run nox session ${{ matrix.nox_session.session }}
run: nox -s "${{ matrix.nox_session.session }}" -v

# Share ./docs/reports so that they can be deployed with doc in next job
- name: Share reports with other jobs
Expand All @@ -77,6 +88,25 @@ jobs:
name: reports_dir
path: ./docs/reports

build_doc:
runs-on: ubuntu-latest
if: github.event_name == 'pull_request'
steps:
- name: Checkout
uses: actions/checkout@v2

- name: Install python 3.9 for nox
uses: actions/setup-python@v4
with:
python-version: 3.9
architecture: x64

- name: Install noxfile requirements
run: pip install -r noxfile-requirements.txt

- name: Build the doc including example gallery
run: nox -s docs -- build

publish_release:
needs: run_all_tests
runs-on: ubuntu-latest
Expand All @@ -87,47 +117,37 @@ jobs:
GITHUB_CONTEXT: ${{ toJSON(github) }}
run: echo "$GITHUB_CONTEXT"

- uses: actions/checkout@v2
- name: Checkout with no depth
uses: actions/checkout@v2
with:
fetch-depth: 0 # so that gh-deploy works

- name: Install python 3.9 for nox
uses: actions/setup-python@v4
with:
python-version: 3.9
architecture: x64

# 1) retrieve the reports generated previously
- name: Retrieve reports
uses: actions/download-artifact@master
with:
name: reports_dir
path: ./docs/reports

# Conda install
- name: Install conda v3.7
uses: conda-incubator/setup-miniconda@v2
with:
# auto-update-conda: true
python-version: 3.7
activate-environment: noxenv
- run: conda info
shell: bash -l {0} # so that conda works
- run: conda list
shell: bash -l {0} # so that conda works

# Nox install
- name: Install noxfile requirements
shell: bash -l {0} # so that conda works
run: pip install -r noxfile-requirements.txt
- run: conda list
shell: bash -l {0} # so that conda works

# 5) Run the flake8 report and badge
- name: Run flake8 analysis and generate corresponding badge
shell: bash -l {0} # so that conda works
run: nox -s flake8

# -------------- only on Ubuntu + MAIN PUSH (no pull request, no tag) -----------

# 5) Publish the doc and test reports
- name: \[not on TAG\] Publish documentation, tests and coverage reports
if: github.event_name == 'push' && startsWith(github.ref, 'refs/heads') # startsWith(matrix.os,'ubuntu')
shell: bash -l {0} # so that conda works
run: nox -s publish

# 6) Publish coverage report
Expand All @@ -151,7 +171,6 @@ jobs:
# 7) Create github release and build the wheel
- name: \[TAG only\] Build wheel and create github release
if: github.event_name == 'push' && startsWith(github.ref, 'refs/tags')
shell: bash -l {0} # so that conda works
run: nox -s release -- ${{ secrets.GITHUB_TOKEN }}

# 8) Publish the wheel on PyPi
Expand Down
2 changes: 1 addition & 1 deletion ci_tools/flake8-requirements.txt
Expand Up @@ -2,7 +2,7 @@ setuptools_scm>=3,<4
flake8>=3.6,<4
flake8-html>=0.4,<1
flake8-bandit>=2.1.1,<3
bandit<1.7.3
bandit<1.7.3 # temporary until this is fixed https://github.com/tylerwince/flake8-bandit/issues/21
flake8-bugbear>=20.1.0,<21.0.0
flake8-docstrings>=1.5,<2
flake8-print>=3.1.1,<4
Expand Down

0 comments on commit 8c2e707

Please sign in to comment.