Skip to content

[3.11] gh-96357: Improve typing.get_overloads coverage (GH-96358)#96371

Merged
JelleZijlstra merged 2 commits into
python:3.11from
miss-islington:backport-675e347-3.11
Sep 29, 2022
Merged

[3.11] gh-96357: Improve typing.get_overloads coverage (GH-96358)#96371
JelleZijlstra merged 2 commits into
python:3.11from
miss-islington:backport-675e347-3.11

Conversation

@miss-islington

@miss-islington miss-islington commented Aug 29, 2022

Copy link
Copy Markdown
Contributor

(cherry picked from commit 675e347)

Co-authored-by: Nikita Sobolev mail@sobolevn.me

(cherry picked from commit 675e347)

Co-authored-by: Nikita Sobolev <mail@sobolevn.me>
@miss-islington

Copy link
Copy Markdown
Contributor Author

@sobolevn and @JelleZijlstra: Status check is done, and it's a success ❌ .

@sobolevn sobolevn closed this Aug 29, 2022
@sobolevn sobolevn reopened this Aug 29, 2022
@miss-islington

Copy link
Copy Markdown
Contributor Author

@sobolevn and @JelleZijlstra: Status check is done, and it's a success ❌ .

1 similar comment
@miss-islington

Copy link
Copy Markdown
Contributor Author

@sobolevn and @JelleZijlstra: Status check is done, and it's a success ❌ .

@JelleZijlstra

Copy link
Copy Markdown
Member

Will merge after 3.11.0 final.

@sobolevn

sobolevn commented Sep 5, 2022

Copy link
Copy Markdown
Member

Offtopic. I am always confused: why status check "is a success", but icon is a red "no-no" sign?
Снимок экрана 2022-09-06 в 0 39 19

Should it be this way?

@JelleZijlstra

Copy link
Copy Markdown
Member

Yes, that feels like a bug. I think there's been some discussion of it in the core dev discord but I don't recall the cause.

@JelleZijlstra JelleZijlstra merged commit f418842 into python:3.11 Sep 29, 2022
@miss-islington miss-islington deleted the backport-675e347-3.11 branch September 29, 2022 01:06
@sobolevn

Copy link
Copy Markdown
Member

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants