Skip to content
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

Get all CPAN distributions using scroll API for MetaCPAN #2817

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

zmughal
Copy link

@zmughal zmughal commented May 18, 2021

  • Get all CPAN distributions using scroll API for MetaCPAN
  • Revert "Remove CPAN from missing task"

Fixes #1961.

irb(main)> PackageManager::CPAN.project_names.size
=> 37945

which is roughly what we get from

$ curl -s 'https://fastapi.metacpan.org/v1/release/_search?q=status:latest'  | jq '.hits.total'
37972

Of note, searching for distributions directly gets:

$ curl -s 'https://fastapi.metacpan.org/v1/distribution/_search'  | jq '.hits.total'
42548

I'm not yet sure about why there is a discrepancy.

@zmughal
Copy link
Author

zmughal commented May 19, 2021

I think I could shorten the code a little bit by using a ternary operator inside the loop. Let me know which approach you prefer!

@zmughal
Copy link
Author

zmughal commented May 28, 2021

@tiegz , @jsonperl, any thoughts? I believe I’ll need to rebase this now.

If this isn’t the right place or the right issue to tackle, let me know.

@zmughal
Copy link
Author

zmughal commented May 30, 2021

Fixed the merge conflict.

@zmughal
Copy link
Author

zmughal commented Jun 25, 2021

Hello! What can I do to get this merged or, if needed, implement similar functionality in another repository?

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.

Getting list of all CPAN package names is broken
1 participant