Skip to content

[flake8-type-checking] Stabilize unquoted-type-alias (TC007) #16638

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 12, 2025

Conversation

ntBre
Copy link
Contributor

@ntBre ntBre commented Mar 11, 2025

Summary

Stabilizes TC007. The test was already in the right place.

Test Plan

No open issues or PRs.

Summary
--

Stabilizes TC007. The test was already in the right place.

Test Plan
--

No open issues or PRs.
@ntBre ntBre added the rule Implementing or modifying a lint rule label Mar 11, 2025
@ntBre ntBre added this to the v0.10 milestone Mar 11, 2025
Copy link

codspeed-hq bot commented Mar 11, 2025

CodSpeed Performance Report

Merging #16638 will degrade performances by 12.88%

Comparing brent/tc007-0.10 (70ed16c) with micha/ruff-0.10 (5ec7c13)

Summary

❌ 1 regressions
✅ 31 untouched benchmarks

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Benchmarks breakdown

Benchmark BASE HEAD Change
red_knot_check_file[incremental] 4.8 ms 5.5 ms -12.88%

Copy link
Contributor

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

@ntBre ntBre mentioned this pull request Mar 12, 2025
2 tasks
@ntBre ntBre merged commit e261f63 into micha/ruff-0.10 Mar 12, 2025
20 of 21 checks passed
@ntBre ntBre deleted the brent/tc007-0.10 branch March 12, 2025 12:08
MichaReiser pushed a commit that referenced this pull request Mar 13, 2025
…16638)

Summary
--

Stabilizes TC007. The test was already in the right place.

Test Plan
--

No open issues or PRs.
MichaReiser pushed a commit that referenced this pull request Mar 13, 2025
…16638)

Summary
--

Stabilizes TC007. The test was already in the right place.

Test Plan
--

No open issues or PRs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rule Implementing or modifying a lint rule
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants