[PR #33] [MERGED] chore: remove code emitting deprecation warnings #63

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

📋 Pull Request Information

Original PR: https://github.com/run-llama/workflows-py/pull/33
Author: @masci
Created: 7/25/2025
Status: Merged
Merged: 7/25/2025
Merged by: @masci

Base: mainHead: massi/fix-warnings


📝 Commits (4)

  • a27b150 Remove deprecated calls to Context.set/get
  • 7bf4eca reorganize tests for better readability, add some more coverage
  • 95ec302 cover context.utils
  • 76925ed Merge branch 'main' into massi/fix-warnings

📊 Changes

11 files changed (+135 additions, -87 deletions)

View changed files

📝 src/workflows/context/context.py (+7 -5)
📝 src/workflows/types.py (+1 -1)
tests/context/__init__.py (+0 -0)
📝 tests/context/test_context.py (+18 -53)
tests/context/test_serializers.py (+31 -0)
tests/context/test_utils.py (+38 -0)
📝 tests/test_resources.py (+8 -1)
📝 tests/test_retry_policy.py (+3 -3)
📝 tests/test_streaming.py (+3 -3)
📝 tests/test_utils.py (+5 -0)
📝 tests/test_workflow.py (+21 -21)

📄 Description

Also in this PR

  • cover some extra lines with low-hanging test cases
  • exclude deprecated code from test coverage
  • reorganize a bit the test code to increase readability

🔄 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/33 **Author:** [@masci](https://github.com/masci) **Created:** 7/25/2025 **Status:** ✅ Merged **Merged:** 7/25/2025 **Merged by:** [@masci](https://github.com/masci) **Base:** `main` ← **Head:** `massi/fix-warnings` --- ### 📝 Commits (4) - [`a27b150`](https://github.com/run-llama/workflows-py/commit/a27b150ddc3bc261143811e7c0fae00fd105bdb3) Remove deprecated calls to Context.set/get - [`7bf4eca`](https://github.com/run-llama/workflows-py/commit/7bf4eca3a1c6df8e526a58beebb242b4e749c3ac) reorganize tests for better readability, add some more coverage - [`95ec302`](https://github.com/run-llama/workflows-py/commit/95ec302f6b2c121f5cdbadbd49d11cd92e9e119b) cover context.utils - [`76925ed`](https://github.com/run-llama/workflows-py/commit/76925edc6fc8b0c40fb73b78006d489b47f06987) Merge branch 'main' into massi/fix-warnings ### 📊 Changes **11 files changed** (+135 additions, -87 deletions) <details> <summary>View changed files</summary> 📝 `src/workflows/context/context.py` (+7 -5) 📝 `src/workflows/types.py` (+1 -1) ➕ `tests/context/__init__.py` (+0 -0) 📝 `tests/context/test_context.py` (+18 -53) ➕ `tests/context/test_serializers.py` (+31 -0) ➕ `tests/context/test_utils.py` (+38 -0) 📝 `tests/test_resources.py` (+8 -1) 📝 `tests/test_retry_policy.py` (+3 -3) 📝 `tests/test_streaming.py` (+3 -3) 📝 `tests/test_utils.py` (+5 -0) 📝 `tests/test_workflow.py` (+21 -21) </details> ### 📄 Description Also in this PR - cover some extra lines with low-hanging test cases - exclude deprecated code from test coverage - reorganize a bit the test code to increase readability --- <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:27 -05:00
yindo closed this issue 2026-02-16 02:16:27 -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#63