ci: disable scheduled integration tests, upgrade setup-python to v5 - #5
Conversation
Remove on.schedule from integration-tests.yml to stop daily failure emails; workflow requires API keys not available in scheduled runs. Upgrade actions/setup-python v4 -> v5 in both workflows to resolve the Node.js 20 deprecation warning on GitHub Actions runners. https://claude.ai/code/session_013oWykq7fEdCyUKgGFbeXpt
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3d33de7dab
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| - uses: actions/checkout@v4 | ||
| - name: Set up Python ${{ matrix.python-version }} | ||
| uses: actions/setup-python@v4 | ||
| uses: actions/setup-python@v5 |
There was a problem hiding this comment.
When these workflows run on current GitHub Actions runners, actions/setup-python@v5 still declares runs.using: node20 in its official action metadata, while the Node 24 migration is in v6 (https://github.com/actions/setup-python#breaking-changes-in-v6). Since this same v5 update was applied in unit-tests.yml, the Node 20 deprecation warning this commit is meant to fix will continue to be emitted for setup-python unless both workflows move to the Node 24 action version.
Useful? React with 👍 / 👎.
2 participants
Changes
2 files changed, 5 lines total. No src/, no pyproject.toml, no uv.lock.
.github/workflows/integration-tests.ymlon.schedulecron trigger; upgradesetup-python@v4→@v5.github/workflows/unit-tests.ymlsetup-python@v4→@v5onlyWhy
integration-tests.ymlran on schedule but requiresANTHROPIC_API_KEY,OPENAI_API_KEY, andLANGSMITH_API_KEYwhich are not available in unattended scheduled runs → every run failed → daily email.actions/setup-python@v4uses Node.js 20 runtime; GitHub Actions now forces it onto Node.js 24 with a deprecation warning.@v5resolves this.Integration tests remain runnable on demand: Actions → Integration Tests → Run workflow.
Supersedes
Closes #2, #3, #4 — all had merge conflicts or mixed in unrelated business code changes.
Generated by Claude Code