Skip to content

remove fetching of ratings from API #3061

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 12, 2024
Merged

remove fetching of ratings from API #3061

merged 1 commit into from
May 12, 2024

Conversation

haarg
Copy link
Member

@haarg haarg commented May 12, 2024

CPANRatings is long dead, and we don't display the old data anywhere.

Removing the API calls fetching the data.

CPANRatings is long dead, and we don't display the old data anywhere.

Removing the API calls fetching the data.
Copy link

codecov bot commented May 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.96%. Comparing base (2be9dab) to head (a672367).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3061      +/-   ##
==========================================
- Coverage   73.02%   72.96%   -0.06%     
==========================================
  Files          74       73       -1     
  Lines        2543     2534       -9     
  Branches      356      356              
==========================================
- Hits         1857     1849       -8     
  Misses        555      555              
+ Partials      131      130       -1     
Files Coverage Δ
lib/MetaCPAN/Web/Model/ReleaseInfo.pm 81.89% <ø> (ø)

... and 1 file with indirect coverage changes

@haarg haarg merged commit 1b801a6 into master May 12, 2024
8 of 9 checks passed
@haarg haarg deleted the haarg/no-rating branch May 12, 2024 09:26
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