Skip to content

T39 injection method - #49

Merged
niden merged 2 commits into
masterfrom
T39-injection-method
Nov 3, 2026
Merged

niden merged 2 commits into
masterfrom
T39-injection-method

Conversation

@niden

@niden niden commented Nov 3, 2026

Copy link
Copy Markdown
Member

Hello!

In raising this pull request, I confirm the following:

  • I have read and understood the Contributing Guidelines
  • I have checked that another pull request for this purpose does not exist
  • I wrote some tests for this PR
  • I have updated the relevant CHANGELOG
  • I have created a PR for the documentation about this change

  • ResponseListener::handle() runs the injection step with a ResponseInterface and no event. The MVC listener calls it, so MVC behavior does not change.

Thanks

@niden niden self-assigned this Nov 3, 2026
@niden niden added the enhancement New feature or request label Nov 3, 2026
@niden
niden requested a review from Jeckerson November 3, 2026 10:38
@github-actions

This comment has been minimized.

@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown

Code Metrics Report

master (5f7c400) #49 (70e332e) +/-
Coverage 100.0% 100.0% 0.0%
Test Execution Time 3s 4s +1s
Details
  |                     | master (5f7c400) | #49 (70e332e) | +/-  |
  |---------------------|------------------|---------------|------|
  | Coverage            |           100.0% |        100.0% | 0.0% |
  |   Files             |               70 |            70 |    0 |
  |   Lines             |             1825 |          1826 |   +1 |
+ |   Covered           |             1825 |          1826 |   +1 |
- | Test Execution Time |               3s |            4s |  +1s |

Code coverage of files in pull request scope (100.0% → 100.0%, patch 100.0%)

Files Coverage +/- Patch Coverage Status
src/DebugBar/ResponseListener.php 100.0% 0.0% 100.0% modified

Reported by octocov

@niden
niden merged commit 05f8fc9 into master Nov 3, 2026
24 checks passed
@niden
niden deleted the T39-injection-method branch November 3, 2026 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant