Skip to content

Commit eba2873

Browse files
committed
[ruff] Support Unicode dummy variable names
1 parent 80520b3 commit eba2873

17 files changed

Lines changed: 92 additions & 19 deletions

‎README.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -266,7 +266,7 @@ fixable = ["ALL"]
266266
unfixable = []
267267

268268
# Allow unused variables when underscore-prefixed.
269-
dummy-variable-rgx = "^(_+|(_+[a-zA-Z0-9_]*[a-zA-Z0-9]+?))$"
269+
dummy-variable-rgx = '^(?:_+|_\p{XID_Continue}*[\p{XID_Continue}--_])$'
270270

271271
[format]
272272
# Like Black, use double quotes for strings.

‎crates/ruff/tests/cli/snapshots/cli__lint__requires_python_extend_from_shared_config.snap‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ linter.explicit_preview_rules = false
8787
linter.extension = ExtensionMapping({})
8888
linter.allowed_confusables = []
8989
linter.builtins = []
90-
linter.dummy_variable_rgx = ^(_+|(_+[a-zA-Z0-9_]*[a-zA-Z0-9]+?))$
90+
linter.dummy_variable_rgx = ^(?:_+|_\p{XID_Continue}*[\p{XID_Continue}--_])$
9191
linter.external = []
9292
linter.ignore_init_module_imports = true
9393
linter.logger_objects = []

‎crates/ruff/tests/cli/snapshots/cli__lint__requires_python_no_tool.snap‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ linter.explicit_preview_rules = false
8989
linter.extension = ExtensionMapping({})
9090
linter.allowed_confusables = []
9191
linter.builtins = []
92-
linter.dummy_variable_rgx = ^(_+|(_+[a-zA-Z0-9_]*[a-zA-Z0-9]+?))$
92+
linter.dummy_variable_rgx = ^(?:_+|_\p{XID_Continue}*[\p{XID_Continue}--_])$
9393
linter.external = []
9494
linter.ignore_init_module_imports = true
9595
linter.logger_objects = []

‎crates/ruff/tests/cli/snapshots/cli__lint__requires_python_no_tool_preview_enabled.snap‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ linter.explicit_preview_rules = false
9191
linter.extension = ExtensionMapping({})
9292
linter.allowed_confusables = []
9393
linter.builtins = []
94-
linter.dummy_variable_rgx = ^(_+|(_+[a-zA-Z0-9_]*[a-zA-Z0-9]+?))$
94+
linter.dummy_variable_rgx = ^(?:_+|_\p{XID_Continue}*[\p{XID_Continue}--_])$
9595
linter.external = []
9696
linter.ignore_init_module_imports = true
9797
linter.logger_objects = []

‎crates/ruff/tests/cli/snapshots/cli__lint__requires_python_no_tool_target_version_override.snap‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ linter.explicit_preview_rules = false
9191
linter.extension = ExtensionMapping({})
9292
linter.allowed_confusables = []
9393
linter.builtins = []
94-
linter.dummy_variable_rgx = ^(_+|(_+[a-zA-Z0-9_]*[a-zA-Z0-9]+?))$
94+
linter.dummy_variable_rgx = ^(?:_+|_\p{XID_Continue}*[\p{XID_Continue}--_])$
9595
linter.external = []
9696
linter.ignore_init_module_imports = true
9797
linter.logger_objects = []

‎crates/ruff/tests/cli/snapshots/cli__lint__requires_python_pyproject_toml_above.snap‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ linter.explicit_preview_rules = false
8888
linter.extension = ExtensionMapping({})
8989
linter.allowed_confusables = []
9090
linter.builtins = []
91-
linter.dummy_variable_rgx = ^(_+|(_+[a-zA-Z0-9_]*[a-zA-Z0-9]+?))$
91+
linter.dummy_variable_rgx = ^(?:_+|_\p{XID_Continue}*[\p{XID_Continue}--_])$
9292
linter.external = []
9393
linter.ignore_init_module_imports = true
9494
linter.logger_objects = []

‎crates/ruff/tests/cli/snapshots/cli__lint__requires_python_pyproject_toml_above_with_tool.snap‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ linter.explicit_preview_rules = false
8989
linter.extension = ExtensionMapping({})
9090
linter.allowed_confusables = []
9191
linter.builtins = []
92-
linter.dummy_variable_rgx = ^(_+|(_+[a-zA-Z0-9_]*[a-zA-Z0-9]+?))$
92+
linter.dummy_variable_rgx = ^(?:_+|_\p{XID_Continue}*[\p{XID_Continue}--_])$
9393
linter.external = []
9494
linter.ignore_init_module_imports = true
9595
linter.logger_objects = []

‎crates/ruff/tests/cli/snapshots/cli__lint__requires_python_ruff_toml_above-2.snap‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ linter.explicit_preview_rules = false
8787
linter.extension = ExtensionMapping({})
8888
linter.allowed_confusables = []
8989
linter.builtins = []
90-
linter.dummy_variable_rgx = ^(_+|(_+[a-zA-Z0-9_]*[a-zA-Z0-9]+?))$
90+
linter.dummy_variable_rgx = ^(?:_+|_\p{XID_Continue}*[\p{XID_Continue}--_])$
9191
linter.external = []
9292
linter.ignore_init_module_imports = true
9393
linter.logger_objects = []

‎crates/ruff/tests/cli/snapshots/cli__lint__requires_python_ruff_toml_above.snap‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ linter.explicit_preview_rules = false
8787
linter.extension = ExtensionMapping({})
8888
linter.allowed_confusables = []
8989
linter.builtins = []
90-
linter.dummy_variable_rgx = ^(_+|(_+[a-zA-Z0-9_]*[a-zA-Z0-9]+?))$
90+
linter.dummy_variable_rgx = ^(?:_+|_\p{XID_Continue}*[\p{XID_Continue}--_])$
9191
linter.external = []
9292
linter.ignore_init_module_imports = true
9393
linter.logger_objects = []

‎crates/ruff/tests/cli/snapshots/cli__lint__requires_python_ruff_toml_no_target_fallback.snap‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ linter.explicit_preview_rules = false
8787
linter.extension = ExtensionMapping({})
8888
linter.allowed_confusables = []
8989
linter.builtins = []
90-
linter.dummy_variable_rgx = ^(_+|(_+[a-zA-Z0-9_]*[a-zA-Z0-9]+?))$
90+
linter.dummy_variable_rgx = ^(?:_+|_\p{XID_Continue}*[\p{XID_Continue}--_])$
9191
linter.external = []
9292
linter.ignore_init_module_imports = true
9393
linter.logger_objects = []

0 commit comments

Comments
 (0)