[PR #204] [MERGED] Update ruff checks #215

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

📋 Pull Request Information

Original PR: https://github.com/run-llama/workflows-py/pull/204
Author: @adrianlyjak
Created: 11/15/2025
Status: Merged
Merged: 11/18/2025
Merged by: @adrianlyjak

Base: mainHead: adrian/ty-ruff


📝 Commits (2)

📊 Changes

96 files changed (+363 additions, -349 deletions)

View changed files

.changeset/odd-cycles-mate.md (+6 -0)
📝 .pre-commit-config.yaml (+1 -1)
📝 examples/agent.ipynb (+2 -2)
📝 examples/client/base/workflow_client.py (+2 -4)
📝 examples/client/base/workflow_server.py (+4 -4)
📝 examples/client/human_in_the_loop/workflow_client_hitl.py (+1 -1)
📝 examples/client/human_in_the_loop/workflow_server_hitl.py (+2 -2)
📝 examples/document_agents/finance_triage_agent.ipynb (+3 -3)
📝 examples/document_processing.ipynb (+6 -6)
📝 examples/durable_workflows.ipynb (+4 -4)
📝 examples/feature_walkthrough.ipynb (+5 -4)
📝 examples/observability/workflow_context_logging.ipynb (+1 -2)
📝 examples/observability/workflows_observability_pt1.ipynb (+9 -9)
📝 examples/observability/workflows_observability_pt2.ipynb (+20 -21)
📝 examples/observability/workflows_observablitiy_arize_phoenix.ipynb (+3 -3)
📝 examples/observability/workflows_observablitiy_langfuse.ipynb (+3 -3)
📝 examples/server/server_example.py (+2 -2)
📝 examples/state_management_with_vector_databases.ipynb (+8 -6)
📝 examples/streaming_internal_events.ipynb (+3 -2)
📝 packages/llama-index-utils-workflow/src/llama_index/utils/workflow/__init__.py (+25 -28)

...and 76 more files

📄 Description

Adds various ruff lint rules (sort imports. removing prints, use union syntax rather than Optional and Union, require function args).

All fixes are automated from ruff. Backs out and keeps some unions in tests, as it seems to fail the 3.9 tests. Also adds some ignores for stricter fixes to notebooks and examples rather than adding a bunch of types


🔄 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/204 **Author:** [@adrianlyjak](https://github.com/adrianlyjak) **Created:** 11/15/2025 **Status:** ✅ Merged **Merged:** 11/18/2025 **Merged by:** [@adrianlyjak](https://github.com/adrianlyjak) **Base:** `main` ← **Head:** `adrian/ty-ruff` --- ### 📝 Commits (2) - [`b7fa95e`](https://github.com/run-llama/workflows-py/commit/b7fa95e144488d62d58355e47f0882c7afecc1c1) Ruff: sort and other stricter rules - [`9f49707`](https://github.com/run-llama/workflows-py/commit/9f497074a8be009b360a73dcd6103caaa14b1cad) Add changesets ### 📊 Changes **96 files changed** (+363 additions, -349 deletions) <details> <summary>View changed files</summary> ➕ `.changeset/odd-cycles-mate.md` (+6 -0) 📝 `.pre-commit-config.yaml` (+1 -1) 📝 `examples/agent.ipynb` (+2 -2) 📝 `examples/client/base/workflow_client.py` (+2 -4) 📝 `examples/client/base/workflow_server.py` (+4 -4) 📝 `examples/client/human_in_the_loop/workflow_client_hitl.py` (+1 -1) 📝 `examples/client/human_in_the_loop/workflow_server_hitl.py` (+2 -2) 📝 `examples/document_agents/finance_triage_agent.ipynb` (+3 -3) 📝 `examples/document_processing.ipynb` (+6 -6) 📝 `examples/durable_workflows.ipynb` (+4 -4) 📝 `examples/feature_walkthrough.ipynb` (+5 -4) 📝 `examples/observability/workflow_context_logging.ipynb` (+1 -2) 📝 `examples/observability/workflows_observability_pt1.ipynb` (+9 -9) 📝 `examples/observability/workflows_observability_pt2.ipynb` (+20 -21) 📝 `examples/observability/workflows_observablitiy_arize_phoenix.ipynb` (+3 -3) 📝 `examples/observability/workflows_observablitiy_langfuse.ipynb` (+3 -3) 📝 `examples/server/server_example.py` (+2 -2) 📝 `examples/state_management_with_vector_databases.ipynb` (+8 -6) 📝 `examples/streaming_internal_events.ipynb` (+3 -2) 📝 `packages/llama-index-utils-workflow/src/llama_index/utils/workflow/__init__.py` (+25 -28) _...and 76 more files_ </details> ### 📄 Description Adds various ruff lint rules (sort imports. removing prints, use union syntax rather than `Optional` and `Union`, require function args). All fixes are automated from ruff. Backs out and keeps some unions in tests, as it seems to fail the 3.9 tests. Also adds some ignores for stricter fixes to notebooks and examples rather than adding a bunch of types --- <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:16:56 -05:00
yindo closed this issue 2026-02-16 02:16:56 -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#215