Skip to content

Enhancement: Extract collectors#1119

Merged
localheinz merged 1 commit intomainfrom
feature/collector
Dec 16, 2025
Merged

Enhancement: Extract collectors#1119
localheinz merged 1 commit intomainfrom
feature/collector

Conversation

@localheinz
Copy link
Copy Markdown
Member

@localheinz localheinz commented Dec 15, 2025

This pull request

  • extracts collectors

Fixes #1103.

@localheinz localheinz self-assigned this Dec 15, 2025
@localheinz localheinz force-pushed the feature/collector branch 2 times, most recently from 4a15d7d to 35ddb44 Compare December 15, 2025 14:41
@codecov
Copy link
Copy Markdown

codecov bot commented Dec 15, 2025

Codecov Report

❌ Patch coverage is 71.38264% with 89 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.56%. Comparing base (4b4dfcc) to head (b8e0eb5).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
...r/PhpTokenTokenizeConstructFromSourceCollector.php 4.22% 68 Missing ⚠️
...lector/TokenGetAllConstructFromSourceCollector.php 85.88% 12 Missing ⚠️
src/Exception/SourceCouldNotBeParsed.php 0.00% 5 Missing ⚠️
.../Collector/DefaultConstructFromFinderCollector.php 90.90% 2 Missing ⚠️
...rc/Collector/DefaultConstructFromFileCollector.php 95.23% 1 Missing ⚠️
...ector/DefaultConstructFromSplFileInfoCollector.php 95.00% 1 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##               main    #1119       +/-   ##
=============================================
- Coverage     92.81%   71.56%   -21.25%     
- Complexity       51       97       +46     
=============================================
  Files             6       17       +11     
  Lines           167      313      +146     
=============================================
+ Hits            155      224       +69     
- Misses           12       89       +77     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@localheinz localheinz marked this pull request as ready for review December 15, 2025 16:33
@localheinz localheinz force-pushed the feature/collector branch 12 times, most recently from 0ef09c4 to 35eb8dc Compare December 16, 2025 08:34
@localheinz localheinz merged commit 32ea658 into main Dec 16, 2025
28 of 30 checks passed
@localheinz localheinz deleted the feature/collector branch December 16, 2025 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update for PHP8+: Utilize PhpToken::tokenize()

2 participants