Skip to content

Commit

Permalink
feat(ci): Update github actions to also test python 3.12 (#1456)
Browse files Browse the repository at this point in the history
* Update github actions to also test python 3.12

* Update setup.py

* Update setup.py

* Update kfp-tekton-unittests.yml

* Update kfp-tekton-unittests.yml

* Update kfp-tekton-unittests.yml

* Update kfp-tekton-unittests.yml

* Update README.md
  • Loading branch information
Tomcli authored Feb 13, 2024
1 parent a9d7df9 commit 9f568f2
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
3 changes: 1 addition & 2 deletions .github/workflows/kfp-tekton-unittests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,14 +9,13 @@ on:

env:
GITHUB_ACTION: "true"
SETUPTOOLS_USE_DISTUTILS: "stdlib"

jobs:
python-unittest:
runs-on: ubuntu-latest
strategy:
matrix:
python-version: ['3.8', '3.9', '3.10', '3.11']
python-version: ['3.8', '3.9', '3.10', '3.11', '3.12']
steps:
- uses: actions/checkout@v4
- name: Set up Python ${{ matrix.python-version }}
Expand Down
2 changes: 1 addition & 1 deletion sdk/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ adding the `TektonCompiler` and the `TektonClient`:

## Project Prerequisites

- Python: `3.8` or later
- Python: `3.8` or later. For Python 3.12, make sure to not have the `SETUPTOOLS_USE_DISTUTILS` flag because it's already [deprecated](https://github.com/pypa/setuptools/issues/4002).
- Tekton: [`v0.53.2`](https://github.com/tektoncd/pipeline/releases/tag/v0.53.2) or [later](https://github.com/tektoncd/pipeline/releases/latest)
- Tekton CLI: [`0.30.1`](https://github.com/tektoncd/cli/releases/tag/v0.30.1)
- Kubeflow Pipelines: [KFP with Tekton backend](/guides/kfp_tekton_install.md)
Expand Down
3 changes: 2 additions & 1 deletion sdk/python/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,8 @@
REQUIRES = [
"kfp>=1.8.10,<1.8.23",
"kfp-tekton-server-api==1.8.0rc8",
"PyYAML>=6,<7"
"PyYAML>=6,<7",
"setuptools>=69.1"
]

TESTS_REQUIRE = [
Expand Down

0 comments on commit 9f568f2

Please sign in to comment.