Skip to content

Treat type(Protocol) et al as metaclass base#12770

Merged
charliermarsh merged 5 commits intomainfrom
charlie/meta
Aug 9, 2024
Merged

Treat type(Protocol) et al as metaclass base#12770
charliermarsh merged 5 commits intomainfrom
charlie/meta

Conversation

@charliermarsh
Copy link
Member

Summary

Closes #12736.

| ["abc", "ABCMeta"]
| ["enum", "EnumMeta" | "EnumType"]
)
}),
Copy link
Member Author

Choose a reason for hiding this comment

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

This is the only logic change.

@charliermarsh charliermarsh added the bug Something isn't working label Aug 9, 2024
@codspeed-hq
Copy link

codspeed-hq bot commented Aug 9, 2024

CodSpeed Performance Report

Merging #12770 will not alter performance

Comparing charlie/meta (f98f274) with main (37b9bac)

Summary

✅ 32 untouched benchmarks

@github-actions
Copy link
Contributor

github-actions bot commented Aug 9, 2024

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

charliermarsh and others added 3 commits August 9, 2024 16:04
Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
@charliermarsh
Copy link
Member Author

Thanks Alex!

@charliermarsh charliermarsh enabled auto-merge (squash) August 9, 2024 20:05
@charliermarsh charliermarsh merged commit 69e1c56 into main Aug 9, 2024
@charliermarsh charliermarsh deleted the charlie/meta branch August 9, 2024 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

N805 False positive for class Meta(type(base))

3 participants