Skip to content

Conversation

dmontagu
Copy link
Contributor

@dmontagu dmontagu commented Feb 15, 2024

I think this fixes the issue reported in #8729 (comment), though it does NOT address #8729.

Either way, we need to add a test.

Copy link

cloudflare-workers-and-pages bot commented Feb 15, 2024

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: b378f7b
Status: ✅  Deploy successful!
Preview URL: https://f6d48996.pydantic-docs2.pages.dev
Branch Preview URL: https://dmontagu-mypy-plugin-fix-exc.pydantic-docs2.pages.dev

View logs

@dmontagu dmontagu added the relnotes-fix Used for bugfixes. label Feb 15, 2024
Copy link

codspeed-hq bot commented Feb 15, 2024

CodSpeed Performance Report

Merging #8826 will not alter performance

Comparing dmontagu/mypy-plugin-fix-excessive-api-deferral (b378f7b) with main (812516d)

Summary

✅ 10 untouched benchmarks

Copy link
Contributor

@sydney-runkle sydney-runkle left a comment

Choose a reason for hiding this comment

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

Awesome, thanks for the fix @dmontagu

@sydney-runkle sydney-runkle merged commit 9fc5e18 into main Feb 22, 2024
@sydney-runkle sydney-runkle deleted the dmontagu/mypy-plugin-fix-excessive-api-deferral branch February 22, 2024 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
relnotes-fix Used for bugfixes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants