Skip to content

Commit

Permalink
Merge pull request #37 from jumpstarter-dev/python-bump
Browse files Browse the repository at this point in the history
Raise minimum python version to 3.11
  • Loading branch information
mangelajo authored Aug 7, 2024
2 parents 2d34996 + 83b06de commit b61ab5c
Show file tree
Hide file tree
Showing 5 changed files with 84 additions and 123 deletions.
21 changes: 20 additions & 1 deletion .github/workflows/pytest.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,13 +10,18 @@ permissions:
contents: read
pull-requests: read
jobs:
pytest:
pytest-matrix:
runs-on: ubuntu-latest
strategy:
matrix:
python-version: ["3.11", "3.12"]
steps:
- uses: actions/checkout@v4

- name: Install Python
uses: actions/setup-python@v4
with:
python-version: ${{ matrix.python-version }}

- name: Install poetry
uses: abatilo/actions-poetry@v2
Expand All @@ -35,3 +40,17 @@ jobs:

- name: Run the automated tests (for example)
run: poetry run pytest

# https://github.com/orgs/community/discussions/26822
pytest:
runs-on: ubuntu-latest
needs: [pytest-matrix]
if: ${{ always() }}
steps:
- run: exit 1
if: >-
${{
contains(needs.*.result, 'failure')
|| contains(needs.*.result, 'cancelled')
|| contains(needs.*.result, 'skipped')
}}
3 changes: 1 addition & 2 deletions jumpstarter/config/client.py
Original file line number Diff line number Diff line change
@@ -1,9 +1,8 @@
import os
from dataclasses import dataclass
from typing import Optional
from typing import Optional, Self

import yaml
from typing_extensions import Self

from .common import CONFIG_API_VERSION, CONFIG_PATH
from .env import JMP_DRIVERS_ALLOW, JMP_ENDPOINT, JMP_TOKEN
Expand Down
3 changes: 1 addition & 2 deletions jumpstarter/config/user.py
Original file line number Diff line number Diff line change
@@ -1,9 +1,8 @@
import os
from dataclasses import dataclass
from typing import Optional
from typing import Optional, Self

import yaml
from typing_extensions import Self

from .client import ClientConfig
from .common import CONFIG_API_VERSION, CONFIG_PATH
Expand Down
Loading

0 comments on commit b61ab5c

Please sign in to comment.