Skip to content

Parallelize the downloads and insertions to database - #34

Merged
sethmlarson merged 9 commits into
sethmlarson:mainfrom
miketheman:miketheman/modernize
Mar 4, 2026
Merged

sethmlarson merged 9 commits into
sethmlarson:mainfrom
miketheman:miketheman/modernize

Conversation

@miketheman

Copy link
Copy Markdown
Contributor

A fair amount in here, please review by commits

It's stable, and easier to parse than HTML

Signed-off-by: Mike Fiedler <miketheman@gmail.com>
Already have fetched it, might as well use it.

Refs: https://docs.pypi.org/api/json/#ownership

Signed-off-by: Mike Fiedler <miketheman@gmail.com>
Instead of `resp`, call it what it is - package data

Signed-off-by: Mike Fiedler <miketheman@gmail.com>
- `re.sub` made `count`  kwarg-only in 3.13
- Use built-in `venv` instead of `virtualenv`
- Remove unneeded parentheses

Signed-off-by: Mike Fiedler <miketheman@gmail.com>
Signed-off-by: Mike Fiedler <miketheman@gmail.com>
Signed-off-by: Mike Fiedler <miketheman@gmail.com>
Signed-off-by: Mike Fiedler <miketheman@gmail.com>
Signed-off-by: Mike Fiedler <miketheman@gmail.com>
Comment thread README.md Outdated

@sethmlarson sethmlarson left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
@sethmlarson sethmlarson changed the title chore: modernize Parallelize the downloads and insertions to database Mar 4, 2026
@sethmlarson
sethmlarson merged commit 3ee0028 into sethmlarson:main Mar 4, 2026
@miketheman
miketheman deleted the miketheman/modernize branch March 4, 2026 21:59
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.

3 participants