[PR #509] [MERGED] chore: remove boilerplate base classes #511

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

📋 Pull Request Information

Original PR: https://github.com/run-llama/llama_deploy/pull/509
Author: @masci
Created: 5/23/2025
Status: Merged
Merged: 5/23/2025
Merged by: @masci

Base: mainHead: massi/cleanup


📝 Commits (4)

📊 Changes

32 files changed (+181 additions, -360 deletions)

View changed files

📝 e2e_tests/message_queues/aws/test_message_queue.py (+2 -1)
📝 e2e_tests/message_queues/conftest.py (+2 -2)
📝 e2e_tests/message_queues/kafka/test_message_queue.py (+2 -1)
📝 e2e_tests/message_queues/rabbitmq/test_message_queue.py (+2 -1)
📝 e2e_tests/message_queues/redis/test_message_queue.py (+2 -1)
📝 e2e_tests/message_queues/simple/test_message_queue.py (+2 -1)
📝 llama_deploy/__init__.py (+0 -3)
📝 llama_deploy/control_plane/base.py (+5 -8)
📝 llama_deploy/control_plane/server.py (+12 -5)
📝 llama_deploy/message_consumers/__init__.py (+2 -3)
llama_deploy/message_consumers/base.py (+0 -54)
📝 llama_deploy/message_consumers/remote.py (+16 -18)
llama_deploy/message_publishers/__init__.py (+0 -3)
llama_deploy/message_publishers/publisher.py (+0 -41)
📝 llama_deploy/message_queues/apache_kafka.py (+4 -6)
📝 llama_deploy/message_queues/aws.py (+4 -6)
📝 llama_deploy/message_queues/base.py (+5 -9)
📝 llama_deploy/message_queues/rabbitmq.py (+4 -6)
📝 llama_deploy/message_queues/redis.py (+4 -6)
📝 llama_deploy/message_queues/simple/client.py (+5 -7)

...and 12 more files

📄 Description

No description provided


🔄 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/llama_deploy/pull/509 **Author:** [@masci](https://github.com/masci) **Created:** 5/23/2025 **Status:** ✅ Merged **Merged:** 5/23/2025 **Merged by:** [@masci](https://github.com/masci) **Base:** `main` ← **Head:** `massi/cleanup` --- ### 📝 Commits (4) - [`880151e`](https://github.com/run-llama/llama_deploy/commit/880151ebe312eefca330c00dee30fb323385dc5f) remove message_publishers boilerplate - [`066cf17`](https://github.com/run-llama/llama_deploy/commit/066cf174d3d2693d60a36c0d79583ad7bc0372ea) move class under tests - [`2563b00`](https://github.com/run-llama/llama_deploy/commit/2563b00443d47a496c827c5e08ad84d954986a5c) remove base class - [`673db56`](https://github.com/run-llama/llama_deploy/commit/673db568a522d25e136a2790cd8c48f109c3ad18) remove base service ### 📊 Changes **32 files changed** (+181 additions, -360 deletions) <details> <summary>View changed files</summary> 📝 `e2e_tests/message_queues/aws/test_message_queue.py` (+2 -1) 📝 `e2e_tests/message_queues/conftest.py` (+2 -2) 📝 `e2e_tests/message_queues/kafka/test_message_queue.py` (+2 -1) 📝 `e2e_tests/message_queues/rabbitmq/test_message_queue.py` (+2 -1) 📝 `e2e_tests/message_queues/redis/test_message_queue.py` (+2 -1) 📝 `e2e_tests/message_queues/simple/test_message_queue.py` (+2 -1) 📝 `llama_deploy/__init__.py` (+0 -3) 📝 `llama_deploy/control_plane/base.py` (+5 -8) 📝 `llama_deploy/control_plane/server.py` (+12 -5) 📝 `llama_deploy/message_consumers/__init__.py` (+2 -3) ➖ `llama_deploy/message_consumers/base.py` (+0 -54) 📝 `llama_deploy/message_consumers/remote.py` (+16 -18) ➖ `llama_deploy/message_publishers/__init__.py` (+0 -3) ➖ `llama_deploy/message_publishers/publisher.py` (+0 -41) 📝 `llama_deploy/message_queues/apache_kafka.py` (+4 -6) 📝 `llama_deploy/message_queues/aws.py` (+4 -6) 📝 `llama_deploy/message_queues/base.py` (+5 -9) 📝 `llama_deploy/message_queues/rabbitmq.py` (+4 -6) 📝 `llama_deploy/message_queues/redis.py` (+4 -6) 📝 `llama_deploy/message_queues/simple/client.py` (+5 -7) _...and 12 more files_ </details> ### 📄 Description _No description provided_ --- <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:15:24 -05:00
yindo closed this issue 2026-02-16 02:15:24 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: run-llama/llama_deploy#511