-
-
Notifications
You must be signed in to change notification settings - Fork 107
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
Hotfix: Restore __init__ method; more robust initialization for singleton locks #338
Conversation
for more information, see https://pre-commit.ci
… init-signature-fix
for more information, see https://pre-commit.ci
… init-signature-fix
for more information, see https://pre-commit.ci
Will add a test for this as well. |
Sorry for this mess. I understand if you want to take longer to review this time; did not realize so much infrastructure relies on this package. |
No worries All good. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks reasonable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add tests for the error message for each parameter differing 👍
I had this test failure on |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Never mind, got it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixes issue with subclassing brought up in pypa/virtualenv#2735 (discussed in #334)