FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

gh-117398: Add datetime C-API type check test for subinterpreters by neonene · Pull Request #119604 · python/cpython · GitHub

/ cpython Public

gh-117398: Add datetime C-API type check test for subinterpreters - #119604

Merged
ericsnowcurrently merged 40 commits into
python:mainfrom
neonene:capi_type_check
Jun 13, 2024
ConversationCommits (40)ChecksFiles changed
Merged

gh-117398: Add datetime C-API type check test for subinterpreters#119604
ericsnowcurrently merged 40 commits into
python:mainfrom
neonene:capi_type_check

Conversation

neonene commented May 27, 2024 •
edited
Loading

Copy link
Copy Markdown
Contributor

Check if the DateType C-API type matches the datetime.date type on main and shared/isolated subinterpreters.

@ericsnowcurrently Please review if this usage is not invalid.

bedevere-app Bot added tests Tests in the Lib/test dir awaiting review labels May 27, 2024
bedevere-app Bot mentioned this pull request May 27, 2024
Closed
ericsnowcurrently requested changes May 27, 2024
View reviewed changes

ericsnowcurrently left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

The test makes sense. Thanks for adding it. I have one question.

Comment thread Modules/_testmultiphase.c Outdated

bedevere-app Bot commented May 27, 2024

Copy link
Copy Markdown

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

neonene requested review from abalkin and pganssle as code owners May 28, 2024 03:20

neonene commented May 28, 2024

Copy link
Copy Markdown
Contributor Author

I have made the requested changes; please review again.

bedevere-app Bot commented May 28, 2024

Copy link
Copy Markdown

Thanks for making the requested changes!

@ericsnowcurrently: please review the changes made to this pull request.

bedevere-app Bot requested a review from ericsnowcurrently May 28, 2024 03:45
neonene changed the title gh-117398: Add test for datetime C API type check gh-117398: Add C-API test for datetime static type check May 28, 2024

ericsnowcurrently commented May 28, 2024

Copy link
Copy Markdown
Member

It looks like part of your PR was removed by mistake. I thought the new test in test_capi.test_misc would still be there.

neonene changed the title gh-117398: Add C-API test for datetime static type check gh-117398: Add datetime C-API type check test for subinterpreters May 29, 2024
bedevere-app Bot requested a review from ericsnowcurrently June 13, 2024 10:28
ericsnowcurrently requested changes Jun 13, 2024
View reviewed changes

ericsnowcurrently left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Thanks for moving that test! There's just one thing to address and then this PR should be ready to merge. Thanks for your patience.

Comment thread Lib/test/support/__init__.py Outdated

bedevere-app Bot commented Jun 13, 2024

Copy link
Copy Markdown

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

neonene commented Jun 13, 2024

Copy link
Copy Markdown
Contributor Author

I have made the requested changes; please review again.

bedevere-app Bot commented Jun 13, 2024

Copy link
Copy Markdown

Thanks for making the requested changes!

@ericsnowcurrently: please review the changes made to this pull request.

bedevere-app Bot requested a review from ericsnowcurrently June 13, 2024 18:00
ericsnowcurrently approved these changes Jun 13, 2024
View reviewed changes

ericsnowcurrently left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM

ericsnowcurrently merged commit 50a3895 into python:main Jun 13, 2024

miss-islington-app Bot commented Jun 13, 2024

Copy link
Copy Markdown

Thanks @neonene for the PR, and @ericsnowcurrently 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 request Jun 13, 2024
pythongh-117398: Add datetime C-API type check test for subinterprete…
8fb3ea8 
…rs (pythongh-119604)

Check if the DateTime C-API type matches the datetime.date type on main and shared/isolated subinterpreters.
(cherry picked from commit 50a3895)

Co-authored-by: neonene <53406459+neonene@users.noreply.github.com>

bedevere-app Bot commented Jun 13, 2024

Copy link
Copy Markdown

GH-120463 is a backport of this pull request to the 3.13 branch.

bedevere-app Bot removed the needs backport to 3.13 bugs and security fixes label Jun 13, 2024

ericsnowcurrently commented Jun 13, 2024

Copy link
Copy Markdown
Member

Thanks again for all your work on this, @neonene!

neonene deleted the capi_type_check branch June 13, 2024 18:10
ericsnowcurrently pushed a commit that referenced this pull request Jun 13, 2024
[3.13] gh-117398: Add datetime C-API type check test for subinterpret…
15c3d00 
…ers (gh-120463)

Check if the DateTime C-API type matches the datetime.date type on main and shared/isolated subinterpreters.

(cherry picked from commit 50a3895, AKA gh-119604)

Co-authored-by: neonene <53406459+neonene@users.noreply.github.com>
freakboy3742 mentioned this pull request Jun 14, 2024
Merged
mrahtz pushed a commit to mrahtz/cpython that referenced this pull request Jun 30, 2024
pythongh-117398: Add datetime C-API type check test for subinterprete…
2129fa4 
…rs (pythongh-119604)

Check if the DateTime C-API type matches the datetime.date type on main and shared/isolated subinterpreters.
neonene mentioned this pull request Jul 10, 2024
Merged
noahbkim pushed a commit to hudson-trading/cpython that referenced this pull request Jul 11, 2024
pythongh-117398: Add datetime C-API type check test for subinterprete…
78a5805 
…rs (pythongh-119604)

Check if the DateTime C-API type matches the datetime.date type on main and shared/isolated subinterpreters.
estyxx pushed a commit to estyxx/cpython that referenced this pull request Jul 17, 2024
pythongh-117398: Add datetime C-API type check test for subinterprete…
6285938 
…rs (pythongh-119604)

Check if the DateTime C-API type matches the datetime.date type on main and shared/isolated subinterpreters.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

ericsnowcurrently ericsnowcurrently approved these changes
markshannon Awaiting requested review from markshannon
gvanrossum Awaiting requested review from gvanrossum
pganssle Awaiting requested review from pganssle pganssle is a code owner
abalkin Awaiting requested review from abalkin

Assignees

No one assigned

Labels

skip news tests Tests in the Lib/test dir

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

Back | FazBrowse Home | New Git URL