Skip to content

Filter backpanned favourites without truncating#1459

Merged
oalders merged 1 commit intomasterfrom
oalders/metacpan-api/backpan-filter
Apr 2, 2026
Merged

Filter backpanned favourites without truncating#1459
oalders merged 1 commit intomasterfrom
oalders/metacpan-api/backpan-filter

Conversation

@oalders
Copy link
Copy Markdown
Member

@oalders oalders commented Apr 2, 2026

This fixes treatment of users who have a long list of favourites and allows us to return a deduplicated list of all favourites rather than a sometimes truncated list.

@oalders oalders force-pushed the oalders/metacpan-api/backpan-filter branch 3 times, most recently from 7f89f44 to 7506eb6 Compare April 2, 2026 14:55
Improve by_user backpan filter

so that we are more likely to get all user favorites rather than just a
portion of them, for users with a lot of favorites.
@oalders oalders force-pushed the oalders/metacpan-api/backpan-filter branch from 7506eb6 to 8d40b97 Compare April 2, 2026 15:27
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 2, 2026

Codecov Report

❌ Patch coverage is 81.25000% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 62.38%. Comparing base (0a7f2f5) to head (8d40b97).
⚠️ Report is 2 commits behind head on master.

Files with missing lines Patch % Lines
lib/MetaCPAN/Query/Favorite.pm 81.25% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1459      +/-   ##
==========================================
+ Coverage   62.16%   62.38%   +0.21%     
==========================================
  Files         137      137              
  Lines        4393     4397       +4     
  Branches      659      660       +1     
==========================================
+ Hits         2731     2743      +12     
+ Misses       1410     1402       -8     
  Partials      252      252              
Files with missing lines Coverage Δ
lib/MetaCPAN/Query/Favorite.pm 50.00% <81.25%> (+21.73%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@oalders oalders merged commit 320d6b6 into master Apr 2, 2026
5 checks passed
@oalders oalders deleted the oalders/metacpan-api/backpan-filter branch April 2, 2026 16:17
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.

1 participant