Skip to content

Commit

Permalink
chore(abt): upgrade python to 3.13 and upgrade dependencies on analys…
Browse files Browse the repository at this point in the history
…es battery (#16430)

# Overview

Upgrade python and dependencies on the analysis battery test.

## Checks

- [x] Works locally
- [x] CI lint works
- [x] CI test works
  • Loading branch information
y3rsh authored Oct 8, 2024
1 parent f8608fd commit 44aa757
Show file tree
Hide file tree
Showing 5 changed files with 330 additions and 369 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/analyses-snapshot-lint.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ jobs:
- name: Setup Python
uses: 'actions/setup-python@v5'
with:
python-version: '3.12'
python-version: '3.13.0-rc.3'
cache: 'pipenv'
cache-dependency-path: analyses-snapshot-testing/Pipfile.lock
- name: Setup
Expand Down
12 changes: 6 additions & 6 deletions .github/workflows/analyses-snapshot-test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -48,8 +48,8 @@ jobs:
ANALYSIS_REF: ${{ github.event.inputs.ANALYSIS_REF || github.head_ref || 'edge' }}
SNAPSHOT_REF: ${{ github.event.inputs.SNAPSHOT_REF || github.head_ref || 'edge' }}
# If we're running because of workflow_dispatch, use the user input to decide
# whether to open a PR on failure. Otherwise, there is no user input, so always
# open a PR on failure.
# whether to open a PR on failure. Otherwise, there is no user input,
# so we only open a PR if the PR has the label 'gen-analyses-snapshot-pr'
OPEN_PR_ON_FAILURE: ${{ (github.event_name == 'workflow_dispatch' && github.events.inputs.OPEN_PR_ON_FAILURE) || ((github.event_name != 'workflow_dispatch') && (contains(github.event.pull_request.labels.*.name, 'gen-analyses-snapshot-pr'))) }}
PR_TARGET_BRANCH: ${{ github.event.pull_request.base.ref || 'not a pr'}}
steps:
Expand All @@ -75,10 +75,10 @@ jobs:
working-directory: analyses-snapshot-testing
run: make build-opentrons-analysis

- name: Set up Python 3.12
- name: Set up Python 3.13
uses: actions/setup-python@v5
with:
python-version: '3.12'
python-version: '3.13.0-rc.3'
cache: 'pipenv'
cache-dependency-path: analyses-snapshot-testing/Pipfile.lock

Expand Down Expand Up @@ -107,7 +107,7 @@ jobs:
- name: Create Snapshot update Request
id: create_pull_request
if: always() && steps.handle_failure.outcome == 'success' && env.OPEN_PR_ON_FAILURE == 'true' && github.event_name == 'pull_request'
uses: peter-evans/create-pull-request@v6
uses: peter-evans/create-pull-request@v7
with:
commit-message: 'fix(analyses-snapshot-testing): heal analyses snapshots'
title: 'fix(analyses-snapshot-testing): heal ${{ env.ANALYSIS_REF }} snapshots'
Expand All @@ -130,7 +130,7 @@ jobs:
- name: Create Snapshot update Request on edge overnight failure
if: always() && steps.handle_failure.outcome == 'success' && github.event_name == 'schedule'
uses: peter-evans/create-pull-request@v6
uses: peter-evans/create-pull-request@v7
with: # scheduled run uses the default values for ANALYSIS_REF and SNAPSHOT_REF which are edge
commit-message: 'fix(analyses-snapshot-testing): heal ${{ env.ANALYSIS_REF }} snapshots'
title: 'fix(analyses-snapshot-testing): heal ${{ env.ANALYSIS_REF }} snapshots'
Expand Down
24 changes: 11 additions & 13 deletions analyses-snapshot-testing/Pipfile
Original file line number Diff line number Diff line change
Expand Up @@ -4,20 +4,18 @@ url = "https://pypi.org/simple"
verify_ssl = true

[packages]
pytest = "==8.1.1"
black = "==24.3.0"
selenium = "==4.19.0"
importlib-metadata = "==7.1.0"
requests = "==2.31.0"
pytest = "==8.3.3"
black = "==24.10.0"
importlib-metadata = "==8.5.0"
httpx = "==0.27.2"
python-dotenv = "==1.0.1"
mypy = "==1.9.0"
types-requests = "==2.31.0.20240311"
rich = "==13.7.1"
pydantic = "==2.6.4"
ruff = "==0.3.4"
docker = "==7.0.0"
syrupy = "==4.6.1"
mypy = "==1.11.2"
rich = "==13.9.2"
pydantic = "==2.9.2"
ruff = "==0.6.9"
docker = "==7.1.0"
syrupy = "==4.7.2"
pytest-html = "==4.1.1"

[requires]
python_version = "3.12"
python_version = "3.13"
Loading

0 comments on commit 44aa757

Please sign in to comment.