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

Appearance settings

bpo-45678: Add functools.singledispatchmethod tests#29328

Merged
ambv merged 2 commits into
python:mainpython/cpython:mainfrom
AlexWaygood:add-functools-testsAlexWaygood/cpython:add-functools-testsCopy head branch name to clipboard
Nov 3, 2021
Merged

bpo-45678: Add functools.singledispatchmethod tests#29328
ambv merged 2 commits into
python:mainpython/cpython:mainfrom
AlexWaygood:add-functools-testsAlexWaygood/cpython:add-functools-testsCopy head branch name to clipboard

Conversation

@AlexWaygood

@AlexWaygood AlexWaygood commented Oct 30, 2021

Copy link
Copy Markdown
Member

@ambv ambv added needs backport to 3.9 needs backport to 3.10 only security fixes 🔨 test-with-buildbots Test PR w/ buildbots; report in status section labels Nov 2, 2021
@bedevere-bot

Copy link
Copy Markdown

🤖 New build scheduled with the buildbot fleet by @ambv for commit 3767676 🤖

If you want to schedule another build, you need to add the ":hammer: test-with-buildbots" label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Nov 2, 2021
@AlexWaygood

Copy link
Copy Markdown
Member Author

All buildbot failures but one are on test_sqlite3. I believe that is unrelated to this PR (see #29382).

The only buildbot that failed on a non-sqlite test was "AMD64 FreeBSD Shared PR", where test_gdb didn't pass. I think that's probably also unrelated? 🙂

@ambv ambv merged commit 5a14929 into python:main Nov 3, 2021
@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @AlexWaygood for the PR, and @ambv for merging it 🌮🎉.. I'm working now to backport this PR to: 3.9, 3.10.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Nov 3, 2021
(cherry picked from commit 5a14929)

Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
@bedevere-bot

Copy link
Copy Markdown

GH-29390 is a backport of this pull request to the 3.10 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.10 only security fixes label Nov 3, 2021
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Nov 3, 2021
(cherry picked from commit 5a14929)

Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
@bedevere-bot

Copy link
Copy Markdown

GH-29391 is a backport of this pull request to the 3.9 branch.

@AlexWaygood AlexWaygood deleted the add-functools-tests branch November 3, 2021 16:13
ambv pushed a commit that referenced this pull request Nov 3, 2021
…H-29390)

(cherry picked from commit 5a14929)

Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
AlexWaygood added a commit to AlexWaygood/cpython that referenced this pull request Nov 3, 2021
…od` bug

This PR fixes a bug in the 3.9 branch where
``functools.singledispatchmethod`` did not properly wrap attributes such as
``__name__``, ``__doc__`` and ``__module__`` of the target method. It also
backports tests already merged into the 3.11 and 3.10 branches in #python#29328 and
python#29390.
@AlexWaygood

Copy link
Copy Markdown
Member Author

As discussed in #29391, the attempted backport of this PR by @miss-islington to Python 3.9 exposed a bug in that branch. A new bugfix/backport PR is here: #29394

ambv added a commit that referenced this pull request Nov 4, 2021
…od` bug (GH-29394)

This PR fixes a bug in the 3.9 branch where
``functools.singledispatchmethod`` did not properly wrap attributes such as
``__name__``, ``__doc__`` and ``__module__`` of the target method. It also
backports tests already merged into the 3.11 and 3.10 branches in #29328 and
#29390.

Co-authored-by: Łukasz Langa <lukasz@langa.pl>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.