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-132527: Added missingw typecode to array() error message#132529

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 5 commits intopython:mainfromChVeen:add-missing-w-typecode
Apr 25, 2025

Conversation

ChVeen
Copy link
Contributor

@ChVeenChVeen commentedApr 14, 2025•
edited by bedevere-appbot
Loading

This PR fixes the missingw in the error message as obtained byarray('x').
It solves issue:#132527

@python-cla-bot
Copy link

python-cla-botbot commentedApr 14, 2025•
edited
Loading

All commit authors signed the Contributor License Agreement.

CLA signed

@bedevere-app
Copy link

Most changes to Pythonrequire a NEWS entry. Add one using theblurb_it web app or theblurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply theskip news label instead.

Copy link
Contributor

@StanFromIrelandStanFromIreland left a comment

Choose a reason for hiding this comment

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

This should get a NEWS entry (click the link in the bot's comment for an easy way to do it) and you need to sign the CLA. I also recommend reading over thedevguide.

@ChVeen
Copy link
ContributorAuthor

This should get a NEWS entry (click the link in the bot's comment for an easy way to do it) and you need to sign the CLA. I also recommend reading over thedevguide.

Done, thanx for your hint :-)

Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>
Copy link
Contributor

@StanFromIrelandStanFromIreland left a comment

Choose a reason for hiding this comment

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

LGTM :-)

@ChVeen

This comment was marked as resolved.

@tomasr8

This comment was marked as resolved.

@StanFromIreland

This comment was marked as resolved.

@ChVeenChVeen marked this pull request as draftApril 22, 2025 09:32
@ChVeenChVeen marked this pull request as ready for reviewApril 22, 2025 09:33
@ChVeen

This comment was marked as resolved.

@StanFromIreland

This comment was marked as resolved.

@ChVeenChVeen closed thisApr 22, 2025
@ChVeenChVeen reopened thisApr 22, 2025
@ChVeen

This comment was marked as resolved.

@StanFromIreland

This comment was marked as resolved.

@ChVeen

This comment was marked as resolved.

@ambv
Copy link
Contributor

If I press this CLA label, I come again to the place where I signed it and it is still signed.

@ChVeen, it is signed for your private @gmail.com address but not for the public @github.com no-reply address:

image

You need to sign for the noreply address, too.

StanFromIreland reacted with thumbs up emoji

@ChVeen
Copy link
ContributorAuthor

If I press this CLA label, I come again to the place where I signed it and it is still signed.

@ChVeen, it is signed for your private @gmail.com address but not for the public @github.com no-reply address:

image

You need to sign for the noreply address, too.

Hi@ambv,
I checked this and it seems I can only choose my private e-mail address from the combobox.
But after choosing this a second time, suddenly a CLA for my public GitHub no-reply address was added as well.
Thanx for your help :-)

tomasr8 reacted with thumbs up emoji

@hugovkhugovk merged commit52454c5 intopython:mainApr 25, 2025
42 checks passed
@hugovkhugovk added the needs backport to 3.13bugs and security fixes labelApr 25, 2025
@miss-islington-app
Copy link

Thanks@ChVeen for the PR, and@hugovk for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull requestApr 25, 2025
…ythonGH-132529)(cherry picked from commit52454c5)Co-authored-by: Christian Veenhuis <124370897+ChVeen@users.noreply.github.com>Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>Co-authored-by: Tomas R. <tomas.roun8@gmail.com>
@bedevere-app
Copy link

GH-132938 is a backport of this pull request to the3.13 branch.

@bedevere-appbedevere-appbot removed the needs backport to 3.13bugs and security fixes labelApr 25, 2025
hugovk pushed a commit that referenced this pull requestApr 25, 2025
…GH-132529) (#132938)Co-authored-by: Christian Veenhuis <124370897+ChVeen@users.noreply.github.com>Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>Co-authored-by: Tomas R. <tomas.roun8@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@tomasr8tomasr8tomasr8 approved these changes

@StanFromIrelandStanFromIrelandStanFromIreland approved these changes

@terryjreedyterryjreedyAwaiting requested review from terryjreedy

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

6 participants
@ChVeen@tomasr8@StanFromIreland@ambv@terryjreedy@hugovk
[8]ページ先頭
©2009-2025 Movatter.jp