Skip to content

ci: support libs with requires-python higher than 3.10 #366

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
james-garner-canonical merged 5 commits into from
Mar 17, 2026
Merged

ci: support libs with requires-python higher than 3.10 #366

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
d8d186e
ci: support libs with requires-python higher than 3.10
james-garner-canonical Mar 17, 2026
295953d
ci: add set -x to complicated bash step for easier debugging in future
james-garner-canonical Mar 17, 2026
17122fa
ci: avoid insecure use of input in changed lines
james-garner-canonical Mar 17, 2026
af671bb
ci: don't bump 3.13 -> 3.14 in this PR since pathops needs a fix for it
james-garner-canonical Mar 17, 2026
e36f2ab
ci: drop odd linebreak
james-garner-canonical Mar 17, 2026
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
10 changes: 7 additions & 3 deletions .github/get-supported-python-versions.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,10 +52,14 @@ def _parse_args() -> pathlib.Path:

def _main(package: pathlib.Path) -> None:
versions = _get_supported_python_versions(package=package)
line = f'versions={json.dumps(versions)}'
print(line)
lines = [
f'versions={json.dumps(versions)}',
f'min_version={versions[0]}',
]
with pathlib.Path(os.environ['GITHUB_OUTPUT']).open('a') as f:
print(line, file=f)
for line in lines:
print(line) # logging
print(line, file=f)


def _get_supported_python_versions(package: pathlib.Path) -> list[str]:
Expand Down
6 changes: 5 additions & 1 deletion .github/workflows/test-package.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,11 +76,15 @@ jobs:
- name: Check substrates needed for integration tests
id: integration-substrates
if: contains(fromJson(steps.tests.outputs.tests), 'integration')
env:
PACKAGE: ${{ inputs.package }}
PYTHON: ${{ steps.python.outputs.min_version }}
run: |
set -x # print lines before executing them
substrates=()
for substrate in k8s machine; do
set +e # don't abort if the next process has a non-zero exit code
uvx --from rust-just just integration-$substrate ${{ inputs.package }} --collect-only -q
uvx --from rust-just just python="$PYTHON" integration-$substrate "$PACKAGE" --collect-only -q
case $? in
0) substrates+=($substrate);;
5) echo "No tests for $substrate";;
Expand Down
Loading