Skip to content

bpo-38431: fix dataclasses.InitVar.__repr__#16702

Merged
serhiy-storchaka merged 4 commits into
python:masterfrom
samuelcolvin:InitVar-fix
Oct 13, 2019
Merged

bpo-38431: fix dataclasses.InitVar.__repr__#16702
serhiy-storchaka merged 4 commits into
python:masterfrom
samuelcolvin:InitVar-fix

Conversation

@samuelcolvin

@samuelcolvin samuelcolvin commented Oct 10, 2019

Copy link
Copy Markdown
Contributor

new version of #16700 based off master.

See original bug pydantic/pydantic#886

https://bugs.python.org/issue38431

Fix repr method of InitVar to work with typing objects.
@brandtbucher brandtbucher added needs backport to 3.8 type-bug An unexpected behavior, bug, or error labels Oct 10, 2019
@brandtbucher

Copy link
Copy Markdown
Member

Thanks for your time @samuelcolvin, and welcome to CPython!

@brandtbucher brandtbucher 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.

This looks like a good change to me!

@brandtbucher

Copy link
Copy Markdown
Member

CC @ericvsmith

@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @samuelcolvin for the PR, and @serhiy-storchaka for merging it 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 13, 2019
pythonGH-16702)

(cherry picked from commit 793cb85)

Co-authored-by: Samuel Colvin <samcolvin@gmail.com>
@bedevere-bot

Copy link
Copy Markdown

GH-16744 is a backport of this pull request to the 3.8 branch.

miss-islington added a commit that referenced this pull request Oct 13, 2019
GH-16702)

(cherry picked from commit 793cb85)

Co-authored-by: Samuel Colvin <samcolvin@gmail.com>
@brandtbucher

Copy link
Copy Markdown
Member

Congrats on your first CPython contribution @samuelcolvin! 🍾

Looking forward to seeing more from you in the future.

jacobneiltaylor pushed a commit to jacobneiltaylor/cpython that referenced this pull request Dec 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type-bug An unexpected behavior, bug, or error

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants