-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
Add Type and Final pandas._typing #26175
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
Comments
|
On second thought it looks too complicated and time consuming for me. Someone else please pick it up. @TomAugspurger maybe you would be interested in it. |
you can pretty much just directly vendor typing_extensions |
@jreback I think you mean copy paste the relevant parts from the source code of typing_extensions. I took a look at their source code but it's a bit complicated for me. I will take a look at it again after a few days when I get some time and see what I can do. |
What benefit do we get from adding Final to these annotations? |
Final would just indicate that a variable is not supposed to be subsequently modified. FWIW I consider it to be a nice to have but not a blocker in the PR where it is mentioned. If we consider bumping min Python version to 3.5.2 then alternately we don’t need to vendor anything |
My preference is to vendor for now. Bumping min Python version, not matter how small, is still a relatively big change to codebase. |
My preference is to just not add these annotations.
… On Apr 23, 2019, at 01:16, gfyoung ***@***.***> wrote:
My preference is to vendor for now. Bumping min Python version, not matter how small, is still a relative big change to codebase.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Final is fine to ignore. I think Type[] is very useful though and that didn't appear until 3.5.2 along with a few (less critical IMO) items: https://docs.python.org/3.5/whatsnew/changelog.html#id28
Have we bumped bug fix releases in the past? We are already dropping Py27 this release so wondering if moving to 3.5.2 from there is adding incremental risk |
I don't recall if we have done incremental bumps like this in the past, but if there is interest, I think that should discussion should be handled separately in another issue / PR. |
Closing this as is. Can discuss move to 3.5.2 in another issue |
Since we are not merging #25975
Type
andFinal
needs to be added topandas._typing
module.Final
is required for PR #25961Type
is required to stay compatible with python 3.5.1 asType
was added in 3.5.2 python docsThe text was updated successfully, but these errors were encountered: