@neralabs/rerail-pack-python 0.1.3 → 0.1.5

package/rules/ruff-sim208-double-negation.json

@@ -3,7 +3,7 @@
   "id": "ruff-sim208-double-negation",
   "revision": 1,
   "mode": "steer",
-  "description": "Port of Ruff's SIM208 (double-negation): `not` applied to another `not` expression, through any parentheses, reported on the outer `not` expression (each outer `not` of a longer chain).",
+  "description": "Remove double negation with not not; use the value itself, or bool() if you need a real boolean.",
   "scope": {
     "include": [
       "**/*"

package/rules/ruff-sim210-if-expr-with-true-false.json

@@ -3,7 +3,7 @@
   "id": "ruff-sim210-if-expr-with-true-false",
   "revision": 1,
   "mode": "steer",
-  "description": "Port of Ruff's SIM210 (if-expr-with-true-false): a conditional expression whose body is the literal `True` and whose `else` arm is `False` (through any parentheses), reported on the whole expression. Ruff's message and fix differ when the test is a comparison or `bool` is shadowed; what is flagged does not.",
+  "description": "Use bool() on the condition instead of a conditional expression that yields True when it holds and False otherwise.",
   "scope": {
     "include": [
       "**/*"

package/rules/ruff-sim211-if-expr-with-false-true.json

@@ -3,7 +3,7 @@
   "id": "ruff-sim211-if-expr-with-false-true",
   "revision": 1,
   "mode": "steer",
-  "description": "Port of Ruff's SIM211 (if-expr-with-false-true): a conditional expression whose body is the literal `False` and whose `else` arm is `True` (through any parentheses), reported on the whole expression.",
+  "description": "Use not on the condition instead of a conditional expression that yields False when it holds and True otherwise.",
   "scope": {
     "include": [
       "**/*"

package/rules/ruff-sim212-if-expr-with-twisted-arms.json

@@ -3,7 +3,7 @@
   "id": "ruff-sim212-if-expr-with-twisted-arms",
   "revision": 1,
   "mode": "steer",
-  "description": "Port of Ruff's SIM212 (if-expr-with-twisted-arms): `B if not A else A` where the negated test operand and the `else` arm are the same plain name (through any parentheses), reported on the whole expression.",
+  "description": "Swap the arms of a conditional expression that tests not x and falls back to x, so the test reads x without the negation.",
   "scope": {
     "include": [
       "**/*"
@@ -81,7 +81,7 @@
       "violation": {
         "anchor": "witness.span",
         "code": "SIM212",
-        "message": "Use `$A if $A else ...` instead of `... if not $A else $A`",
+        "message": "Swap the arms to read `$A if $A else ...`, dropping the `not` from `... if not $A else $A`.",
         "suppress": {
           "noqa": {
             "codes": [

package/rules/ruff-sim300-yoda-conditions.json

@@ -1,9 +1,9 @@
 {
   "format": "rerail.policy/v1",
   "id": "ruff-sim300-yoda-conditions",
-  "revision": 1,
+  "revision": 2,
   "mode": "steer",
-  "description": "Port of Ruff's SIM300 (yoda-conditions): a comparison with one `==`, `!=`, `<`, `<=`, `>` or `>=` whose left operand is more constant-like than its right, on Ruff's three levels (`ConstantLikelihood`, flake8_simplify/rules/yoda_conditions.rs:76-136): literals (not f- or t-strings) are definite; a name or attribute whose name is cased uppercase is probable; tuples, lists and dicts (keys and values, `**` values), binary operations and unary `+`/`-`/`~` take the least of their parts (empty is definite); everything else, sets included, is unlikely. Reported on the comparison.",
+  "description": "Put the variable first and the constant or literal second in comparisons, avoiding Yoda conditions; it reads the way you would say it.",
   "scope": {
     "include": [
       "**/*"
@@ -473,7 +473,7 @@
       "violation": {
         "anchor": "witness.span",
         "code": "SIM300",
-        "message": "Yoda condition detected",
+        "message": "Put the variable first; reverse `<`/`>` and `<=`/`>=` when swapping operands, and keep equality operators unchanged.",
         "suppress": {
           "noqa": {
             "codes": [

package/rules/ruff-sim905-split-static-string.json

@@ -3,7 +3,7 @@
   "id": "ruff-sim905-split-static-string",
   "revision": 1,
   "mode": "steer",
-  "description": "Port of Ruff's SIM905 (split-static-string): `.split(...)` or `.rsplit(...)` called on a str literal (implicit concatenations included; not bytes, f- or t-strings), where `sep` (keyword, else first positional argument) is absent, `None` or a str literal, and `maxsplit` (keyword, else second positional argument) is absent or an integer literal, optionally negated, that fits in an i32 (`get_maxsplit_value`). Reported on the call.",
+  "description": "Use a list literal instead of calling split() or rsplit() on a string literal; the result is known in advance.",
   "scope": {
     "include": [
       "**/*"

package/rules/ruff-sim910-dict-get-with-none-default.json

@@ -3,7 +3,7 @@
   "id": "ruff-sim910-dict-get-with-none-default",
   "revision": 1,
   "mode": "steer",
-  "description": "Port of Ruff's SIM910 (dict-get-with-none-default): `.get(key, None, ...)` whose second positional argument is `None`, with no keywords, called on a dict display or dict comprehension (through parentheses), or on a name whose binding is a dict (Ruff's `is_dict` through `resolves.binding`: a `def`'s `**kwargs`, a dict-typed value or `dict(...)`, a `dict`/`Dict` annotation or return annotation; unpacking carries no value).",
+  "description": "Drop the redundant None default from dict.get() calls; get() already returns None for a missing key.",
   "scope": {
     "include": [
       "**/*"

package/rules/ruff-t100-debugger.json

@@ -3,7 +3,7 @@
   "id": "ruff-t100-debugger",
   "revision": 1,
   "mode": "steer",
-  "description": "Port of Ruff's T100 (debugger): imports of debugger modules (`is_debugger_import`) and of debugger functions (`from M import n` with `M.n` in `is_debugger_call`), once per imported name, matched by module with `resolves.imported`, relative imports by the module's text (`relative: \"text\"`: Ruff ignores the level). It also flags calls whose callee resolves to a debugger function (`resolve_qualified_name` through the file's imports, or the builtin `breakpoint`).",
+  "description": "Remove debugger imports and calls such as pdb.set_trace() and breakpoint() before committing.",
   "scope": {
     "include": [
       "**/*"
@@ -106,7 +106,7 @@
       "violation": {
         "anchor": "witness.span",
         "code": "T100",
-        "message": "Import for a debugger found",
+        "message": "Remove this debugger import of `$D` before committing.",
         "each": "D",
         "suppress": {
           "noqa": {
@@ -183,7 +183,7 @@
       "violation": {
         "anchor": "witness.span",
         "code": "T100",
-        "message": "Trace found: debugger used",
+        "message": "Remove this debugger call before committing.",
         "suppress": {
           "noqa": {
             "codes": [