Skip to content

Omit ASC in ORDER BY for sort fields mapped to expressions - #147

Merged
vjik merged 1 commit into
masterfrom
enh-sort
Oct 21, 2026
Merged

vjik merged 1 commit into
masterfrom
enh-sort

Conversation

@vjik

@vjik vjik commented Oct 20, 2026

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

@vjik
vjik requested a review from a team October 20, 2026 14:04
@vjik vjik added the status:code review The pull request needs review. label Oct 20, 2026
@codecov

codecov Bot commented Oct 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.23%. Comparing base (2c9fc63) to head (13e99e9).

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #147      +/-   ##
============================================
+ Coverage     97.72%   99.23%   +1.51%     
  Complexity       92       92              
============================================
  Files            22       22              
  Lines           264      263       -1     
============================================
+ Hits            258      261       +3     
+ Misses            6        2       -4     

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

@vjik
vjik merged commit 798e710 into master Oct 21, 2026
99 checks passed
@vjik
vjik deleted the enh-sort branch October 21, 2026 07:46
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.

2 participants