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

Search should use underlying repo #51

Open
srossross opened this issue Jan 7, 2025 · 7 comments · May be fixed by #53
Open

Search should use underlying repo #51

srossross opened this issue Jan 7, 2025 · 7 comments · May be fixed by #53
Assignees

Comments

@srossross
Copy link
Contributor

moving away from the in-advance scraper model, the search feature should be broken out into pypi,conda, npm search pages
that search the underlying repo

@karamba228
Copy link

@srossross
Copy link
Contributor Author

yes, that looks good

@karamba228 karamba228 linked a pull request Jan 14, 2025 that will close this issue
@karamba228
Copy link

@srossross please take a look at the linked PR, I did some testing and provided my test results along with it

@srossross
Copy link
Contributor Author

srossross commented Jan 14, 2025

@karamba228 each search should 'search the underlying repo' eg the pypi search should fetch from pypi.org

@karamba228
Copy link

So we are not using the DuckDB tables anymore?

@srossross
Copy link
Contributor Author

no, we are not

@karamba228
Copy link

karamba228 commented Jan 20, 2025

@srossross the PR is updated. For pypi packages, I had to use Libraries.io because pypi API does not support search. So it does not return a list of packages with the similar name. As for the npm and conda, they reach out directly to their respective APIs.

with libraries.io you need an API key to create requests. you will need to add a environment variable to the .env file LIBRARIES_IO_API_KEY

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 a pull request may close this issue.

2 participants