feat: add daily CI workflow to test integrations against latest dependency versions - #123
feat: add daily CI workflow to test integrations against latest dependency versions#123wiliyam wants to merge 2 commits into
Conversation
maldoinc
left a comment
There was a problem hiding this comment.
It's probably best to rework this to reuse the existing tox definitions instead of duplicating them into this yaml workflow.
|
|
||
| - name: Install wireup + latest integration deps | ||
| run: | | ||
| uv pip install --system -e ".[dev]" |
There was a problem hiding this comment.
dev is a dependency group here, not an extra.
| - name: Install wireup + latest integration deps | ||
| run: | | ||
| uv pip install --system -e ".[dev]" | ||
| uv pip install --system --upgrade ${{ matrix.package }} |
There was a problem hiding this comment.
The matrix doesn't install all the required dependencies. For example, the Django job upgrades only Django, but the current tox env also needs djangorestframework, django-ninja, and typing_extensions.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
Thanks for the review @maldoinc — reworked in `3f9f78c` to address all three points:
The `packages` field still lists what we want upgraded to latest (which is the whole point of the daily job — pin-ignore upgrade). Happy to drop it entirely and always upgrade all installed deps if you'd prefer — let me know. |
|
Looks good overall. The only thing missing is the typer integration but other than that happy to merge. |
Summary
Closes #121
Adds
.github/workflows/daily_latest_integrations.yml— a scheduled job that runs the integration test suite daily against the latest released versions of each supported integration package.What it does
workflow_dispatch)fail-fast: false