Skip to content

Cleanup process handling after dropping temp-file handling#5772

Merged
sebastianbergmann merged 1 commit intosebastianbergmann:10.5from
staabm:cleanup
Mar 28, 2024
Merged

Cleanup process handling after dropping temp-file handling#5772
sebastianbergmann merged 1 commit intosebastianbergmann:10.5from
staabm:cleanup

Conversation

@staabm
Copy link
Copy Markdown
Contributor

@staabm staabm commented Mar 28, 2024

after #5766 we can remove some bits of unused code now.

Comment thread src/Util/PHP/DefaultPhpProcess.php
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.15%. Comparing base (500149c) to head (56ff3c9).

Additional details and impacted files
@@             Coverage Diff              @@
##               10.5    #5772      +/-   ##
============================================
+ Coverage     90.14%   90.15%   +0.01%     
+ Complexity     6443     6439       -4     
============================================
  Files           680      679       -1     
  Lines         19543    19537       -6     
============================================
- Hits          17617    17614       -3     
+ Misses         1926     1923       -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@staabm staabm marked this pull request as ready for review March 28, 2024 09:35
@sebastianbergmann sebastianbergmann merged commit 51e0bee into sebastianbergmann:10.5 Mar 28, 2024
@sebastianbergmann sebastianbergmann added the feature/process-isolation Issues related to running tests in separate PHP processes label Mar 28, 2024
@staabm staabm deleted the cleanup branch March 28, 2024 09:48
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Apr 18, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

feature/process-isolation Issues related to running tests in separate PHP processes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants