Skip to content

sort favorites case insensitively #1463

Merged
oalders merged 2 commits intomasterfrom
oalders/metacpan-api/sort-order
Apr 7, 2026
Merged

sort favorites case insensitively #1463
oalders merged 2 commits intomasterfrom
oalders/metacpan-api/sort-order

Conversation

@oalders
Copy link
Copy Markdown
Member

@oalders oalders commented Apr 7, 2026

  • Allow for case-insensitive sort on favorites
  • default to using less RAM for elasticsearch-test container

We will need to re-index faves:

curl -X POST 'https://[production-url]/cpan_v1_01_favorite/_update_by_query?refresh'

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.66%. Comparing base (3d29507) to head (06da407).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1463      +/-   ##
==========================================
+ Coverage   62.62%   62.66%   +0.03%     
==========================================
  Files         137      137              
  Lines        4407     4409       +2     
  Branches      661      662       +1     
==========================================
+ Hits         2760     2763       +3     
+ Misses       1394     1393       -1     
  Partials      253      253              
Files with missing lines Coverage Δ
lib/MetaCPAN/Query/Favorite.pm 66.07% <100.00%> (+3.10%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@oalders oalders force-pushed the oalders/metacpan-api/sort-order branch 2 times, most recently from 7d805c5 to 06da407 Compare April 7, 2026 17:30
mickeyn
mickeyn previously approved these changes Apr 7, 2026
@oalders oalders force-pushed the oalders/metacpan-api/sort-order branch from 06da407 to 806fa49 Compare April 7, 2026 18:31
@oalders oalders merged commit e0eb3db into master Apr 7, 2026
2 of 3 checks passed
@oalders oalders deleted the oalders/metacpan-api/sort-order branch April 7, 2026 18:38
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