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

Concurrent pypi lookups with --update-all #102

Merged
merged 2 commits into from
Dec 13, 2018

Conversation

peterbe
Copy link
Owner

@peterbe peterbe commented Dec 10, 2018

Fixes #101

@peterbe
Copy link
Owner Author

peterbe commented Dec 10, 2018

@mythmon r?

Copy link
Contributor

@mythmon mythmon left a comment

Choose a reason for hiding this comment

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

This seems to work better than I expected. I was expecting the hybrid sync/async to be really messy, but it worked out ok in practice.

hashin.py Outdated
@@ -690,6 +721,12 @@ def get_parser():
action="store_true",
default=False,
)
parser.add_argument(
"--synchronous",
help=("Do not download from pypi in parallel."),
Copy link
Contributor

Choose a reason for hiding this comment

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

Extra parens?

Copy link
Owner Author

Choose a reason for hiding this comment

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

I'm too lazy to find or file the bug for Black :)

Copy link
Owner Author

Choose a reason for hiding this comment

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

For the record, I think the string used to be much longer and when I edited the string, Black didn't remove the bracket now when it all fit on one line.

@@ -1,3 +1,3 @@
flake8==3.5.0
flake8==3.6.0
Copy link
Contributor

Choose a reason for hiding this comment

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

Do you want this update in this patch?

Copy link
Owner Author

Choose a reason for hiding this comment

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

Yes and no. This project is more than a protype/side-project but I can take solace in that the change is unimportant in context of things.
Thanks for checking though!

@peterbe peterbe merged commit 795534b into master Dec 13, 2018
@peterbe peterbe deleted the 101-concurrent-pypi-lookups-with-update-all branch December 13, 2018 01:25
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