Skip to content

Conversation

@ntBre
Copy link
Contributor

@ntBre ntBre commented Mar 12, 2025

Summary

Stabilizes RUF048 and moves its test to the right place. The docs look good.

Test Plan

0 closed or open issues. There was 1 PR related to an extension to the rule, but it was closed without comment.

Summary
--

Stabilizes RUF048 and moves its test to the right place. The docs look good.

Test Plan
--

0 closed or open issues. There was 1 [PR] related to an extension to the rule,
but it was closed without comment.

[PR]: #14701
@ntBre ntBre added the rule Implementing or modifying a lint rule label Mar 12, 2025
@ntBre ntBre added this to the v0.10 milestone Mar 12, 2025
@codspeed-hq
Copy link

codspeed-hq bot commented Mar 12, 2025

CodSpeed Performance Report

Merging #16653 will degrade performances by 10.81%

Comparing brent/ruf048-0.10 (0d01cda) with micha/ruff-0.10 (b8f1284)

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.9 ms 5.5 ms -10.81%

@github-actions
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 57bf330 into micha/ruff-0.10 Mar 12, 2025
20 of 21 checks passed
@ntBre ntBre deleted the brent/ruf048-0.10 branch March 12, 2025 12:34
MichaReiser pushed a commit that referenced this pull request Mar 13, 2025
Summary
--

Stabilizes RUF048 and moves its test to the right place. The docs look
good.

Test Plan
--

0 closed or open issues. There was 1 [PR] related to an extension to the
rule, but it was closed without comment.

[PR]: #14701
MichaReiser pushed a commit that referenced this pull request Mar 13, 2025
Summary
--

Stabilizes RUF048 and moves its test to the right place. The docs look
good.

Test Plan
--

0 closed or open issues. There was 1 [PR] related to an extension to the
rule, but it was closed without comment.

[PR]: #14701
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.

3 participants