Skip to content

Conversation

ntBre
Copy link
Contributor

@ntBre ntBre commented Jun 6, 2025

Summary

Stabilizes the UP049 rule (private-type-parameter) by moving it from Preview to Stable.

UP049 detects and fixes the use of private type parameters (those with leading underscores) in PEP 695 generic classes and functions.

Test plan

  • Verified that UP049 tests pass: crates/ruff_linter/src/rules/pyupgrade/mod.rs
  • Ran full test suite with make test
  • Confirmed that no test migration was needed as UP049 was already in the main rules test function

Rule documentation

https://docs.astral.sh/ruff/rules/private-type-parameter/

@ntBre ntBre added the rule Implementing or modifying a lint rule label Jun 6, 2025
@ntBre ntBre changed the base branch from main to brent/release-0.12.0 June 6, 2025 21:27
@ntBre
Copy link
Contributor Author

ntBre commented Jun 6, 2025

These with the funny descriptions are being generated by claude with a lot of hand-holding. I'm double-checking that the tests are in the right place and that the docs look okay manually, but not necessarily before opening the PRs.

Copy link
Contributor

github-actions bot commented Jun 6, 2025

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

@dylwil3
Copy link
Collaborator

dylwil3 commented Jun 6, 2025

Ran full test suite with make test

lol Claude, I don't think you did...

@ntBre
Copy link
Contributor Author

ntBre commented Jun 6, 2025

I actually do have a Makefile and told it to run make test 😆

@dylwil3
Copy link
Collaborator

dylwil3 commented Jun 6, 2025

I actually do have a Makefile and told it to run make test 😆

oh whoops! sorry, Claude - my mistake!

@ntBre
Copy link
Contributor Author

ntBre commented Jun 6, 2025

Looks like the ecosystem check is all unrelated (claude had the base branch wrong initially again). The docs and tests look good to me.

@ntBre ntBre mentioned this pull request Jun 6, 2025
2 tasks
@ntBre ntBre force-pushed the brent/release-0.12.0 branch 3 times, most recently from 9252447 to 829acf4 Compare June 9, 2025 00:22
@ntBre ntBre force-pushed the brent/stabilize-up049 branch from 650247d to 4ea8d76 Compare June 9, 2025 13:11
@ntBre ntBre merged commit d4d2985 into brent/release-0.12.0 Jun 9, 2025
34 checks passed
@ntBre ntBre deleted the brent/stabilize-up049 branch June 9, 2025 13:54
@ntBre ntBre added this to the v0.12 milestone Jun 10, 2025
ntBre added a commit that referenced this pull request Jun 10, 2025
## Summary
Stabilizes the UP049 rule (private-type-parameter) by moving it from
Preview to Stable.

UP049 detects and fixes the use of private type parameters (those with
leading underscores) in PEP 695 generic classes and functions.

## Test plan
- Verified that UP049 tests pass:
`crates/ruff_linter/src/rules/pyupgrade/mod.rs`
- Ran full test suite with `make test` 
- Confirmed that no test migration was needed as UP049 was already in
the main `rules` test function

## Rule documentation
https://docs.astral.sh/ruff/rules/private-type-parameter/
ntBre added a commit that referenced this pull request Jun 11, 2025
## Summary
Stabilizes the UP049 rule (private-type-parameter) by moving it from
Preview to Stable.

UP049 detects and fixes the use of private type parameters (those with
leading underscores) in PEP 695 generic classes and functions.

## Test plan
- Verified that UP049 tests pass:
`crates/ruff_linter/src/rules/pyupgrade/mod.rs`
- Ran full test suite with `make test` 
- Confirmed that no test migration was needed as UP049 was already in
the main `rules` test function

## Rule documentation
https://docs.astral.sh/ruff/rules/private-type-parameter/
ntBre added a commit that referenced this pull request Jun 17, 2025
## Summary
Stabilizes the UP049 rule (private-type-parameter) by moving it from
Preview to Stable.

UP049 detects and fixes the use of private type parameters (those with
leading underscores) in PEP 695 generic classes and functions.

## Test plan
- Verified that UP049 tests pass:
`crates/ruff_linter/src/rules/pyupgrade/mod.rs`
- Ran full test suite with `make test` 
- Confirmed that no test migration was needed as UP049 was already in
the main `rules` test function

## Rule documentation
https://docs.astral.sh/ruff/rules/private-type-parameter/
ntBre added a commit that referenced this pull request Jun 17, 2025
## Summary
Stabilizes the UP049 rule (private-type-parameter) by moving it from
Preview to Stable.

UP049 detects and fixes the use of private type parameters (those with
leading underscores) in PEP 695 generic classes and functions.

## Test plan
- Verified that UP049 tests pass:
`crates/ruff_linter/src/rules/pyupgrade/mod.rs`
- Ran full test suite with `make test` 
- Confirmed that no test migration was needed as UP049 was already in
the main `rules` test function

## Rule documentation
https://docs.astral.sh/ruff/rules/private-type-parameter/
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