Skip to content

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

Merged
JelleZijlstra merged 2 commits intopython: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 intopython:3.11from
miss-islington:backport-675e347-3.11

Conversation

@miss-islington
Copy link
Copy Markdown
Contributor

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

(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
Copy link
Copy Markdown
Member

sobolevn commented Sep 5, 2022

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