-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Update remaining versions for third-party stubs #6094
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
Conversation
Also remove the python2 markers of packages that don't list Python 2 as supported in the latest version.
Depends on typeshed-internal/stub_uploader#30 |
stubs/dataclasses/METADATA.toml
Outdated
@@ -1 +1 @@ | |||
version = "0.1" | |||
version = "0" |
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.
Is this a typo or intentional?
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.
Intentional, and unfortunately necessary until #6095 is done. This was actually what spawned that issue. Otherwise, the tests try to install dataclasses==0.1.*
, which doesn't exist.
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.
Can we now set this to 0.*
?
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.
I would actually set this to 0.1
0.8
, since 0.9
might contain API changes. I'll send a PR that adds stars to all other versions first.
Co-authored-by: Akuli <[email protected]>
Let's do #6146 first, then this should tie up all remaining bits and bobs. |
@Akuli I hope I have now addressed all issues. A thorough final review would be appreciated! |
The latest version available on PyPI is 0.8, but for some reason that is uninstallable.
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.
One small nit.
Co-authored-by: Akuli <[email protected]>
Also remove the python2 markers of packages that don't list Python 2 as supported in the latest version.
This depends on #6093, which should be merged into this branch before merging to verify CI.