Skip to content

Improve psalm types + Improve tests - #1096

Merged
vjik merged 7 commits into
masterfrom
range-parser
Nov 15, 2025
Merged

vjik merged 7 commits into
masterfrom
range-parser

Conversation

@vjik

@vjik vjik commented Nov 13, 2025 •

Copy link
Copy Markdown
Member
Q A
Is bugfix? ❌
New feature? ❌
Breaks BC? ❌

Related PR: yiisoft/db-pgsql#307

@codecov

codecov Bot commented Nov 13, 2025 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.73%. Comparing base (a740e5f) to head (67c3e26).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #1096   +/-   ##
=========================================
  Coverage     98.73%   98.73%           
  Complexity     1649     1649           
=========================================
  Files           119      119           
  Lines          4276     4276           
=========================================
  Hits           4222     4222           
  Misses           54       54           

☔ 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.

Comment thread src/Schema/Column/AbstractColumnFactory.php
Comment thread src/Schema/Column/ColumnFactoryInterface.php
@vjik vjik changed the title Improve psalm types Improve psalm types + Improve tests Nov 15, 2025
@vjik
vjik requested a review from a team November 15, 2025 12:05
@vjik vjik added the status:code review The pull request needs review. label Nov 15, 2025
@vjik
vjik requested review from Tigrov and samdark November 15, 2025 12:05
@vjik
vjik merged commit 4aa2999 into master Nov 15, 2025
68 checks passed
@vjik
vjik deleted the range-parser branch November 15, 2025 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status:code review The pull request needs review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants