Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cleanup process handling after dropping temp-file handling #5772

Merged
merged 1 commit into from Mar 28, 2024

Conversation

staabm
Copy link
Contributor

@staabm staabm commented Mar 28, 2024

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

Copy link

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
31 checks passed
@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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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.

None yet

2 participants