[PR #317] [MERGED] chore: CI/testing improvements for parallel test execution #320

Closed
opened 2026-02-16 02:17:13 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/run-llama/workflows-py/pull/317
Author: @adrianlyjak
Created: 2/2/2026
Status: Merged
Merged: 2/2/2026
Merged by: @adrianlyjak

Base: mainHead: adrian/workflows-dev


📝 Commits (6)

  • fc1f7ed chore: CI/testing improvements for parallel test execution
  • a96efb1 clean up workflows-dev tests
  • bb928f7 Move workflows-dev to root, and rename to dev
  • a4a4e73 fixing tests
  • 05b737d oh claude
  • 2186841 make pass

📊 Changes

36 files changed (+2549 additions, -555 deletions)

View changed files

📝 .github/workflows/publish_openapi.yml (+1 -1)
📝 .github/workflows/test.yml (+47 -10)
📝 .github/workflows/update_debugger_assets.yml (+1 -1)
📝 .pre-commit-config.yaml (+1 -25)
📝 AGENTS.md (+22 -21)
📝 CONTRIBUTING.md (+10 -2)
📝 packages/llama-index-integration-tests/pyproject.toml (+29 -2)
📝 packages/llama-index-integration-tests/tests/conftest.py (+39 -1)
packages/llama-index-integration-tests/tests/test_docker_postgres.py (+18 -0)
📝 packages/llama-index-utils-workflow/pyproject.toml (+10 -1)
📝 packages/llama-index-workflows-client/pyproject.toml (+15 -0)
📝 packages/llama-index-workflows-server/pyproject.toml (+17 -0)
📝 packages/llama-index-workflows-server/tests/server/test_idle_release.py (+1 -1)
📝 packages/llama-index-workflows-server/tests/server/test_keyed_lock.py (+1 -1)
📝 packages/llama-index-workflows/pyproject.toml (+8 -0)
packages/workflows-dev/README.md (+0 -1)
packages/workflows-dev/pyproject.toml (+0 -31)
packages/workflows-dev/src/workflows_dev/__init__.py (+0 -10)
packages/workflows-dev/tests/test_workflows_dev_cli.py (+0 -325)
📝 pyproject.toml (+37 -11)

...and 16 more files

📄 Description

  • Rename workflows-dev to just dev for easier typing. Move it to the workspace root to make that less vacant. Makes sense as the default package for monorepo management
  • Add pytest-xdist to all the packages
  • Add timeouts to all the packages pytest commands to prevent test hangs
  • Use module-scoped async fixtures for future compatibility with dbos (I think initializes from active loop)
  • Remove mypy in favor of basedpyright/ty type checking
  • Add test-docker CI job for integration tests with testcontainers
  • Exclude integration tests from Python 3.9/3.14 in CI matrix (3.9 for future dbos, and 3.14, as those are run by the test-docker job)

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/run-llama/workflows-py/pull/317 **Author:** [@adrianlyjak](https://github.com/adrianlyjak) **Created:** 2/2/2026 **Status:** ✅ Merged **Merged:** 2/2/2026 **Merged by:** [@adrianlyjak](https://github.com/adrianlyjak) **Base:** `main` ← **Head:** `adrian/workflows-dev` --- ### 📝 Commits (6) - [`fc1f7ed`](https://github.com/run-llama/workflows-py/commit/fc1f7ed00b7068d1b2bfb0d9cccf961677e9ea8c) chore: CI/testing improvements for parallel test execution - [`a96efb1`](https://github.com/run-llama/workflows-py/commit/a96efb14a9949f0aac729beaa6102957944ec174) clean up workflows-dev tests - [`bb928f7`](https://github.com/run-llama/workflows-py/commit/bb928f7871d10f11105465f61ce4314f2dc3a362) Move workflows-dev to root, and rename to dev - [`a4a4e73`](https://github.com/run-llama/workflows-py/commit/a4a4e732a89880f8dce04a7226c376206d3b850e) fixing tests - [`05b737d`](https://github.com/run-llama/workflows-py/commit/05b737d53d848c7877db6b4e03e1d135a0b68a45) oh claude - [`2186841`](https://github.com/run-llama/workflows-py/commit/21868417e704b17626929aed88d28db1ad200205) make pass ### 📊 Changes **36 files changed** (+2549 additions, -555 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/publish_openapi.yml` (+1 -1) 📝 `.github/workflows/test.yml` (+47 -10) 📝 `.github/workflows/update_debugger_assets.yml` (+1 -1) 📝 `.pre-commit-config.yaml` (+1 -25) 📝 `AGENTS.md` (+22 -21) 📝 `CONTRIBUTING.md` (+10 -2) 📝 `packages/llama-index-integration-tests/pyproject.toml` (+29 -2) 📝 `packages/llama-index-integration-tests/tests/conftest.py` (+39 -1) ➕ `packages/llama-index-integration-tests/tests/test_docker_postgres.py` (+18 -0) 📝 `packages/llama-index-utils-workflow/pyproject.toml` (+10 -1) 📝 `packages/llama-index-workflows-client/pyproject.toml` (+15 -0) 📝 `packages/llama-index-workflows-server/pyproject.toml` (+17 -0) 📝 `packages/llama-index-workflows-server/tests/server/test_idle_release.py` (+1 -1) 📝 `packages/llama-index-workflows-server/tests/server/test_keyed_lock.py` (+1 -1) 📝 `packages/llama-index-workflows/pyproject.toml` (+8 -0) ➖ `packages/workflows-dev/README.md` (+0 -1) ➖ `packages/workflows-dev/pyproject.toml` (+0 -31) ➖ `packages/workflows-dev/src/workflows_dev/__init__.py` (+0 -10) ➖ `packages/workflows-dev/tests/test_workflows_dev_cli.py` (+0 -325) 📝 `pyproject.toml` (+37 -11) _...and 16 more files_ </details> ### 📄 Description - Rename `workflows-dev` to just `dev` for easier typing. Move it to the workspace root to make that less vacant. Makes sense as the default package for monorepo management - Add pytest-xdist to all the packages - Add timeouts to all the packages pytest commands to prevent test hangs - Use module-scoped async fixtures for future compatibility with dbos (I think initializes from active loop) - Remove mypy in favor of basedpyright/ty type checking - Add test-docker CI job for integration tests with testcontainers - Exclude integration tests from Python 3.9/3.14 in CI matrix (3.9 for future dbos, and 3.14, as those are run by the test-docker job) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-16 02:17:13 -05:00
yindo closed this issue 2026-02-16 02:17:13 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: run-llama/workflows-py#320