Conversation
vjik
commented
Mar 13, 2026
| Q | A |
|---|---|
| Is bugfix? | ❌ |
| New feature? | ✔️ |
| Breaks BC? | ❌ |
| Tests pass? | ✔️ |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #263 +/- ##
============================================
- Coverage 98.71% 98.54% -0.17%
- Complexity 350 360 +10
============================================
Files 46 47 +1
Lines 932 964 +32
============================================
+ Hits 920 950 +30
- Misses 12 14 +2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
Adds a new QueueFactoryProvider implementation to the queue provider subsystem, allowing queues to be created directly from DI/definition config (via yiisoft/definitions + yiisoft/factory) similarly to the existing adapter-based provider.
Changes:
- Introduce
QueueFactoryProviderthat instantiates and cachesQueueInterfaceinstances from definitions. - Add unit tests covering basic get/has behavior, caching, enum names, container wiring, and invalid configuration handling.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
src/Provider/QueueFactoryProvider.php |
New provider that builds queues from definitions using StrictFactory and normalizes queue names. |
tests/Unit/Provider/QueueFactoryProviderTest.php |
Unit tests validating provider behavior and error cases. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>