Skip to content

Add CompositeExpression#1057

Merged
vjik merged 11 commits intomasterfrom
composite-expression
Sep 4, 2025
Merged

Add CompositeExpression#1057
vjik merged 11 commits intomasterfrom
composite-expression

Conversation

@vjik
Copy link
Member

@vjik vjik commented Sep 4, 2025

Q A
Is bugfix?
New feature? ✔️
Breaks BC?

@vjik vjik requested a review from a team September 4, 2025 12:11
@vjik vjik added the status:code review The pull request needs review. label Sep 4, 2025
@codecov
Copy link

codecov bot commented Sep 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.40%. Comparing base (13e8b4d) to head (5ff68cc).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #1057   +/-   ##
=========================================
  Coverage     98.40%   98.40%           
- Complexity     1634     1639    +5     
=========================================
  Files           115      117    +2     
  Lines          4323     4333   +10     
=========================================
+ Hits           4254     4264   +10     
  Misses           69       69           

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

Copy link
Member

@Tigrov Tigrov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you also add doc here https://github.com/yiisoft/db/blob/master/docs/guide/en/expressions/expressions.md
to better understand how to use it

@vjik vjik requested a review from Tigrov September 4, 2025 13:14
@vjik vjik requested a review from Tigrov September 4, 2025 16:15
@vjik vjik merged commit ce0d534 into master Sep 4, 2025
68 checks passed
@vjik vjik deleted the composite-expression branch September 4, 2025 16:36
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