Skip to content

Switch to shared staged deploy workflows - #123

Merged
m-messer merged 1 commit into
mainfrom
new-workflows
Nov 10, 2026
Merged

m-messer merged 1 commit into
mainfrom
new-workflows

Conversation

@m-messer

@m-messer m-messer commented Nov 9, 2026

Copy link
Copy Markdown
Member

Step 1 of the Python 3.12 upgrade. This PR only changes workflows. A follow-up PR moves the image from baseevalutionfunctionlayer:main-3.8 to main-3.12.

The legacy test-and-deploy.yml (Python 3.8, actions v1/v2, pushes straight to the staging ECR repo) is replaced with the four shared workflows that survey and the other evaluation functions use:

  • test-lint.yml: runs on PRs, Python 3.12, lint only (the pytest steps are removed because the tests call model providers)
  • staging-deploy.yml: on push to main or manual dispatch, a lint job, then evaluation-function-workflows/.github/workflows/deploy.yml@main
  • production-deploy.yml: manual dispatch with version bump; run-database-tests: false (LLM-backed)
  • pre_production_tests.yml: manual Database-Testing run (not used for LLM functions)
  • pytest.ini

Why a separate PR: GitHub only allows workflow_dispatch for workflows that exist on the default branch. Merging this first lets the 3.12 branch be deployed to staging and tested before it's merged.

Note: merging this triggers a staging deploy of the current (3.8) code through the new pipeline.

🤖 Generated with Claude Code

Replace the legacy test-and-deploy.yml with the four workflows used by survey and the
other evaluation functions (test-lint, staging-deploy, production-deploy,
pre_production_tests), calling evaluation-function-workflows deploy.yml@main.
LLM-backed function: CI runs lint only (tests call model providers) and production
deploys keep run-database-tests: false.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@m-messer

m-messer commented Nov 9, 2026

Copy link
Copy Markdown
Member Author

The Test (3.12) failure is expected. The new lint job installs app/requirements.txt on Python 3.12, and main still has the 3.8-only pins (typing==3.7.4.3, numpy==1.24.2, streamlit==1.20.0). The stacked #124 fixes these and passes CI. Merging this PR anyway is fine: its staging deploy will be skipped because the lint job fails, and #124 then gets deployed to staging by manual dispatch.

🤖 Generated with Claude Code

@m-messer
m-messer merged commit bd0739b into main Nov 10, 2026
1 check failed
@m-messer
m-messer deleted the new-workflows branch November 10, 2026 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant