Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

gh-80480: Emit DeprecationWarning for array's 'u' type code#95760

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
hugovk merged 14 commits intopython:mainfromhugovk:80480-deprecate-array-u
Jun 11, 2023

Conversation

hugovk
Copy link
Member

@hugovkhugovk commentedAug 7, 2022•
edited
Loading

The "u" type code forarray was deprecated in Python 3.3 but only via documentation:https://docs.python.org/3.10/library/array.html

Let's add theDeprecationWarning.

It was originally planned to be removed in Python 4.0, but that was before we knew if 3.10 or 4.0 would come after 3.9:

'4.0' is a stand-in for 'sometime after 2.7.final', scheduled for Jan 2020.

#80480 (comment)

So as this is going into3.12 3.13, let's target3.143.15 3.16 for removal.

asedeno reacted with thumbs down emoji
@hugovkhugovk added the stdlibPython modules in the Lib dir labelAug 7, 2022
@asedeno
Copy link

As a user of the'u' array type, I'd like to see this postponed until a UCS4'w' array type is in place to migrate to, as was proposed in#80480 (comment).

@smontanaro
Copy link
Contributor

(I'm working my way through some PRs which have been approved and are labeled "awaiting merge", hence my seemingly bolt from the blue comment. Why?Read here.)

Looks like this was approved before@asedeno's comment. Can someone respond to the expressed concern? (It seems no'w' code has been implemented yet.)

@hugovk
Copy link
MemberAuthor

Thanks for the ping. I commented in the issue, let's continue there.

@hauntsaninjahauntsaninja marked this pull request as draftDecember 23, 2022 05:46
@arhadthedevarhadthedev added extension-modulesC modules in the Modules dir and removed stdlibPython modules in the Lib dir labelsMay 5, 2023
@hugovk
Copy link
MemberAuthor

Updated to target removal in 3.16, following addition of'w' in#105242 and following the plan at#80480 (comment).

@hugovkhugovk marked this pull request as ready for reviewJune 4, 2023 18:01
Copy link
Member

@vstinnervstinner left a comment

Choose a reason for hiding this comment

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

LGTM. I just left minor remarks on the doc. It's up to you to address them or ignore them ;-) Thanks for the updates.

@hugovkhugovk merged commitcc87948 intopython:mainJun 11, 2023
@hugovkhugovk deleted the 80480-deprecate-array-u branchJune 11, 2023 09:17
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@methanemethanemethane approved these changes

@ezio-melottiezio-melottiezio-melotti left review comments

@vstinnervstinnervstinner approved these changes

Assignees
No one assigned
Labels
extension-modulesC modules in the Modules dir
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

8 participants
@hugovk@asedeno@smontanaro@vstinner@methane@ezio-melotti@arhadthedev@bedevere-bot
[8]ページ先頭
©2009-2025 Movatter.jp