Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
52 commits
Select commit Hold shift + click to select a range
921fce8
add python 3.12 to GA testing infrastructure
valeriupredoi Oct 11, 2023
0467388
comment out cartopy
valeriupredoi Oct 11, 2023
c02215c
comment out cartopy
valeriupredoi Oct 11, 2023
0f065a5
comment out cfunits
valeriupredoi Oct 11, 2023
410d267
comment out cfunits
valeriupredoi Oct 11, 2023
901299b
comment out iris
valeriupredoi Oct 11, 2023
43ce46f
comment out iris
valeriupredoi Oct 11, 2023
e649ff8
comment out iris-esmf-regrid
valeriupredoi Oct 11, 2023
924db49
pin fiona
valeriupredoi Oct 11, 2023
4717b66
comment out fiona
valeriupredoi Oct 11, 2023
5e1225e
comment out fiona
valeriupredoi Oct 11, 2023
79d6c2f
Merge branch 'main' into python312
valeriupredoi Oct 31, 2023
4f4e0d1
revert commented out packages
valeriupredoi Oct 31, 2023
a7624b6
revert commented out packages
valeriupredoi Oct 31, 2023
d1d8430
turn off flake8 temporarily
valeriupredoi Nov 1, 2023
cadb20e
align mocker test to python 3.12
valeriupredoi Nov 1, 2023
a0dfff4
Merge branch 'main' into python312
valeriupredoi Nov 6, 2023
302d5b1
remake call to flake8
valeriupredoi Nov 7, 2023
0b1c222
pin flake8
valeriupredoi Nov 7, 2023
1309b49
pin flake8
valeriupredoi Nov 7, 2023
4c69d70
Merge branch 'main' into python312
valeriupredoi Nov 23, 2023
244f622
unpin flake8
valeriupredoi Nov 23, 2023
edd1d7f
unpin flake8
valeriupredoi Nov 23, 2023
7acde28
recomment out flake8 runs
valeriupredoi Nov 23, 2023
93a3cd8
Merge branch 'main' into python311
valeriupredoi Jan 12, 2024
fb1b4d6
unpin upper Python
valeriupredoi Jan 12, 2024
f79c4ec
retrigger flake8
valeriupredoi Jan 12, 2024
7a025e0
ignore prospector for now
valeriupredoi Jan 12, 2024
501e59d
pin flake8
valeriupredoi Jan 15, 2024
4e04603
pin flake8
valeriupredoi Jan 15, 2024
57fcea4
fix new flake8 actual error
valeriupredoi Jan 15, 2024
0c15221
Merge branch 'main' into python312
valeriupredoi Jan 18, 2024
837f323
Merge branch 'main' into python312
valeriupredoi Feb 20, 2024
fa91665
Merge branch 'main' into python312
valeriupredoi Mar 26, 2024
2356581
Merge branch 'main' into python312
valeriupredoi Apr 5, 2024
909bace
inspect environment
valeriupredoi Apr 5, 2024
cbc9d2c
fix inadvertent change
valeriupredoi Apr 8, 2024
1566c2b
readd space after comma - flak 7 is complaining
valeriupredoi Apr 8, 2024
a8063bb
Merge branch 'main' into python312
valeriupredoi Apr 9, 2024
d20c478
Merge branch 'main' into python312
valeriupredoi Apr 22, 2024
c2fd98b
Merge branch 'main' into python312
valeriupredoi May 17, 2024
bc20955
Merge branch 'main' into python312
valeriupredoi May 27, 2024
dcf2dc6
Attempt at supporting Python 3.12
bouweandela May 28, 2024
069af7e
Merge branch 'python-3.12' into python312
valeriupredoi May 28, 2024
4aa944e
reinstate prospector in setup.py
valeriupredoi May 28, 2024
487cdcd
remove prospector pin
valeriupredoi May 28, 2024
09ae656
Remove prospector from setup.py altogether
valeriupredoi May 28, 2024
a30df92
pop a conda list examine env before pip install
valeriupredoi May 28, 2024
62fec6f
unrun GA
valeriupredoi May 28, 2024
78c8bc3
Remove prospector and instead install separate linters
bouweandela May 28, 2024
d6cf2de
run ga one last time
valeriupredoi May 28, 2024
80325ce
unrun ga one last time
valeriupredoi May 28, 2024
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
4 changes: 2 additions & 2 deletions .github/workflows/build-and-deploy-on-pypi.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,10 +22,10 @@ jobs:
- uses: actions/checkout@v4
with:
fetch-depth: 0
- name: Set up Python 3.11
- name: Set up Python 3.12
uses: actions/setup-python@v4
with:
python-version: "3.11"
python-version: "3.12"
- name: Install pep517
run: >-
python -m
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/create-condalock-file.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ jobs:
with:
auto-update-conda: true
activate-environment: esmvaltool-fromlock
python-version: "3.11"
python-version: "3.12"
miniforge-version: "latest"
miniforge-variant: Mambaforge
use-mamba: true
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/install-from-conda.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ jobs:
runs-on: "ubuntu-latest"
strategy:
matrix:
python-version: ["3.9", "3.10", "3.11"]
python-version: ["3.9", "3.10", "3.11", "3.12"]
# fail-fast set to False allows all other tests
# in the workflow to run regardless of any fail
fail-fast: false
Expand Down Expand Up @@ -75,7 +75,7 @@ jobs:
runs-on: "macos-latest"
strategy:
matrix:
python-version: ["3.9", "3.10", "3.11"]
python-version: ["3.9", "3.10", "3.11", "3.12"]
architecture: ["x64"] # need to force Intel, arm64 builds have issues
fail-fast: false
name: OSX Python ${{ matrix.python-version }}
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/install-from-condalock-file.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ jobs:
runs-on: "ubuntu-latest"
strategy:
matrix:
python-version: ["3.9", "3.10", "3.11"]
python-version: ["3.9", "3.10", "3.11", "3.12"]
fail-fast: false
name: Linux Python ${{ matrix.python-version }}
steps:
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/install-from-pypi.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ jobs:
runs-on: "ubuntu-latest"
strategy:
matrix:
python-version: ["3.9", "3.10", "3.11"]
python-version: ["3.9", "3.10", "3.11", "3.12"]
# fail-fast set to False allows all other tests
# in the workflow to run regardless of any fail
fail-fast: false
Expand Down Expand Up @@ -77,7 +77,7 @@ jobs:
runs-on: "macos-latest"
strategy:
matrix:
python-version: ["3.9", "3.10", "3.11"]
python-version: ["3.9", "3.10", "3.11", "3.12"]
architecture: ["x64"] # need to force Intel, arm64 builds have issues
fail-fast: false
name: OSX Python ${{ matrix.python-version }}
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/install-from-source.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ jobs:
runs-on: "ubuntu-latest"
strategy:
matrix:
python-version: ["3.9", "3.10", "3.11"]
python-version: ["3.9", "3.10", "3.11", "3.12"]
fail-fast: false
name: Linux Python ${{ matrix.python-version }}
steps:
Expand Down Expand Up @@ -74,7 +74,7 @@ jobs:
runs-on: "macos-latest"
strategy:
matrix:
python-version: ["3.9", "3.10", "3.11"]
python-version: ["3.9", "3.10", "3.11", "3.12"]
architecture: ["x64"] # need to force Intel, arm64 builds have issues
fail-fast: false
name: OSX Python ${{ matrix.python-version }}
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/run-tests-monitor.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ jobs:
runs-on: "ubuntu-latest"
strategy:
matrix:
python-version: ["3.9", "3.10", "3.11"]
python-version: ["3.9", "3.10", "3.11", "3.12"]
fail-fast: false
name: Linux Python ${{ matrix.python-version }}
steps:
Expand Down Expand Up @@ -55,7 +55,7 @@ jobs:
runs-on: "macos-latest"
strategy:
matrix:
python-version: ["3.9", "3.10", "3.11"]
python-version: ["3.9", "3.10", "3.11", "3.12"]
architecture: ["x64"] # need to force Intel, arm64 builds have issues
fail-fast: false
name: OSX Python ${{ matrix.python-version }}
Expand Down
6 changes: 4 additions & 2 deletions .github/workflows/run-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ jobs:
runs-on: "ubuntu-latest"
strategy:
matrix:
python-version: ["3.9", "3.10", "3.11"]
python-version: ["3.9", "3.10", "3.11", "3.12"]
fail-fast: false
name: Linux Python ${{ matrix.python-version }}
steps:
Expand All @@ -57,6 +57,7 @@ jobs:
- run: mkdir -p test_linux_artifacts_python_${{ matrix.python-version }}
- run: conda --version 2>&1 | tee test_linux_artifacts_python_${{ matrix.python-version }}/conda_version.txt
- run: python -V 2>&1 | tee test_linux_artifacts_python_${{ matrix.python-version }}/python_version.txt
- run: conda list
- run: pip install -e .[develop] 2>&1 | tee test_linux_artifacts_python_${{ matrix.python-version }}/install.txt
- run: conda list
- run: flake8
Expand All @@ -72,7 +73,7 @@ jobs:
runs-on: "macos-latest"
strategy:
matrix:
python-version: ["3.9", "3.10", "3.11"]
python-version: ["3.9", "3.10", "3.11", "3.12"]
architecture: ["x64"] # need to force Intel, arm64 builds have issues
fail-fast: false
name: OSX Python ${{ matrix.python-version }}
Expand All @@ -93,6 +94,7 @@ jobs:
- run: conda --version 2>&1 | tee test_osx_artifacts_python_${{ matrix.python-version }}/conda_version.txt
- run: python -V 2>&1 | tee test_osx_artifacts_python_${{ matrix.python-version }}/python_version.txt
- run: mamba install -c conda-forge git
- run: conda list
- run: pip install -e .[develop] 2>&1 | tee test_osx_artifacts_python_${{ matrix.python-version }}/install.txt
- run: conda list
- run: flake8
Expand Down
10 changes: 7 additions & 3 deletions environment.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ dependencies:
- psutil
- py-cordex
- pybtex
- python >=3.9,<3.12
- python >=3.9
- python-stratify >=0.3
- pyyaml
- requests
Expand All @@ -50,14 +50,14 @@ dependencies:
- sphinx >=6.1.3
- pydata-sphinx-theme
# Python packages needed for testing
- flake8
- mypy >=0.990
- pytest >=3.9,!=6.0.0rc1,!=6.0.0
- pytest-cov >=2.10.1
- pytest-env
- pytest-html !=2.1.0
- pytest-metadata >=1.5.1
- pytest-mock
- pytest-mypy
- pytest-xdist
# Not on conda-forge - ESMValTool_sample_data==0.0.3
# Still for testing, MyPy library stubs
Expand All @@ -68,7 +68,11 @@ dependencies:
- docformatter
- isort
- pre-commit
- prospector >=1.9.0
- pylint
- flake8 >= 7
- pydocstyle
# Not on conda forge - vprof
- yamllint
- yapf
- pip:
- ESMValTool_sample_data
2 changes: 1 addition & 1 deletion esmvalcore/_task.py
Original file line number Diff line number Diff line change
Expand Up @@ -566,7 +566,7 @@ def _run(self, input_files):

returncode = None

with resource_usage_logger(process.pid, self.resource_log),\
with resource_usage_logger(process.pid, self.resource_log), \
open(self.log, 'ab') as log:
last_line = ['']
while returncode is None:
Expand Down
6 changes: 4 additions & 2 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@
],
# Test dependencies
'test': [
'flake8',
'flake8>=7.0.0', # not to pick up E231
'pytest>=3.9,!=6.0.0rc1,!=6.0.0',
'pytest-cov>=2.10.1',
'pytest-env',
Expand Down Expand Up @@ -94,8 +94,10 @@
'codespell',
'docformatter',
'isort',
'flake8>=7',
'pre-commit',
'prospector[with_pyroma]>=1.9.0',
'pylint',
'pydocstyle',
'vprof',
'yamllint',
'yapf',
Expand Down
2 changes: 1 addition & 1 deletion tests/unit/main/test_main.py
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ def test_run_recipe_error(mocker, caplog):
print(caplog.text)

# Check that the exit status is 1
assert exit_.called_once_with(1)
exit_.assert_called_once_with(1)

# Check that only the RecipeError is logged above DEBUG level
errors = [r for r in caplog.records if r.levelname != 'DEBUG']
Expand Down