Stabilize os-symlink (PTH211) by ntBre · Pull Request #20229 · astral-sh/ruff (original) (raw)
Summary
Rule and test/snapshot updated, the docs look good
My one hesitation here is that we could hold off stabilizing the rule until its fix is also ready for stabilization, but this is also the only preview PTH rule, so I think it's okay to stabilize the rule and later (probably in the next minor release) stabilize the fixes together.
Test Plan
Stabilize Stabilize PTH211os-symlink (PTH211)
Implementing or modifying a lint rule
label
2 tasks
ntBre added a commit that referenced this pull request
Summary
Rule and test/snapshot updated, the docs look good
My one hesitation here is that we could hold off stabilizing the rule until its fix is also ready for stabilization, but this is also the only preview PTH rule, so I think it's okay to stabilize the rule and later (probably in the next minor release) stabilize the fixes together.
ntBre added a commit that referenced this pull request
Summary
Rule and test/snapshot updated, the docs look good
My one hesitation here is that we could hold off stabilizing the rule until its fix is also ready for stabilization, but this is also the only preview PTH rule, so I think it's okay to stabilize the rule and later (probably in the next minor release) stabilize the fixes together.
ntBre added a commit that referenced this pull request
Summary
Rule and test/snapshot updated, the docs look good
My one hesitation here is that we could hold off stabilizing the rule until its fix is also ready for stabilization, but this is also the only preview PTH rule, so I think it's okay to stabilize the rule and later (probably in the next minor release) stabilize the fixes together.
ntBre added a commit that referenced this pull request
Summary
Rule and test/snapshot updated, the docs look good
My one hesitation here is that we could hold off stabilizing the rule until its fix is also ready for stabilization, but this is also the only preview PTH rule, so I think it's okay to stabilize the rule and later (probably in the next minor release) stabilize the fixes together.
dcreager added a commit that referenced this pull request
- main: (26 commits)
Ignore deprecated rules unless selected by exact code (#20167)
Stabilize adding future import via config option (#20277)
[
flake8-errmsg] Stabilize extendingraw-string-in-exception(EM101) to support byte strings (#20273) Stabilize the remaining Airflow rules (#20250) [flake8-bugbear] Stabilize support for non-context-manager calls inassert-raises-exception(B017) (#20274) [flake8-commas] Stabilize support for trailing comma checks in type parameter lists (COM812,COM819) (#20275) [pygrep_hooks] Stabilize usingAsyncMockmethods ininvalid-mock-access(PGH005) (#20272) Stabilize new strategy for classifying imports as first party (#20268) [pylint] Stabilize ignoring__init__.pyforuseless-import-alias(PLC0414) (#20271) [pylint] Stabilize adding U+061C tobidirectional-unicode(PLE2502) (#20276) [flake8-simplify] Stabilize fix safety ofmultiple-with-statements(SIM117) (#20270) Stabilizepytest-raises-ambiguous-pattern(RUF043) (#20253) Stabilizef-string-number-format(FURB116) (#20247) [pyupgrade] Removenon-pep604-isinstance(UP038) (#19156) [pandas-vet] Removepandas-df-variable-name(PD901) (#19223) Remove deprecated macOS config file discovery (#19210) Stabilizeredundant-none-literal(PYI061) (#20236) Stabilizegeneric-not-last-base-class(PYI059) (#20246) Stabilizeuseless-class-metaclass-type(UP050) (#20230) Stabilizeos-symlink(PTH211) (#20229) ...
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
[ Show hidden characters]({{ revealButtonHref }})