Skip to content

Clean up page/document arguments as page already has access to the document#482

Merged
PrinsFrank merged 1 commit into
mainfrom
clean-up-page-document-arguments
Jul 24, 2026
Merged

Clean up page/document arguments as page already has access to the document#482
PrinsFrank merged 1 commit into
mainfrom
clean-up-page-document-arguments

Conversation

@PrinsFrank

Copy link
Copy Markdown
Owner

No description provided.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 52.19%. Comparing base (96f507a) to head (8309d1d).

Files with missing lines Patch % Lines
...entStream/PositionedText/PositionedTextElement.php 0.00% 6 Missing ⚠️
src/Extraction/Text/TextExtractor.php 0.00% 3 Missing ⚠️
src/Document/Object/Decorator/Page.php 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##               main     #482   +/-   ##
=========================================
  Coverage     52.19%   52.19%           
  Complexity     1890     1890           
=========================================
  Files           108      108           
  Lines          3548     3548           
=========================================
  Hits           1852     1852           
  Misses         1696     1696           

☔ View full report in Codecov by Harness.
📢 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.

@PrinsFrank
PrinsFrank merged commit 3d76b32 into main Jul 24, 2026
22 checks passed
@PrinsFrank
PrinsFrank deleted the clean-up-page-document-arguments branch July 24, 2026 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants