declint-core 1.0.0

Config and regex linting engine for declint: YAML rule files, message templates, violations. No LSP dependencies.
Documentation 
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95

# declint preset: Python
#
# A curated starter ruleset — the rules from doc/python-tutorial.md.
# Import it from your config:
#
#   import:
#     - preset:python
#
# Presets never declare `languages` — set that in your own config:
#
#   languages: [python]
#
# Every rule here can be overridden where it collides is not supported:
# ids must be unique, so to customize a rule, copy it out of the preset
# and adjust the copy.

version: 1

rules:
  - id: no-tabs
    pattern: '(?m)^\t+'
    message: "PEP 8: spaces, not tabs, for indentation"
    severity: warning
    tests:
      - name: tab indentation is flagged
        text: "\tindented"
        violations: 1
      - name: clean lines pass
        text: "indented"
        violations: 0

  - id: no-semicolons
    pattern: '(?m)^\s*[^#\s].*;\s*$'
    message: "PEP 8: avoid semicolons to separate statements"
    severity: warning

  - id: no-wildcard-imports
    pattern: '(?m)^\s*from\s+[\w.]+\s+import\s+\*'
    message: "wildcard imports make names ambiguous"
    severity: warning

  - id: bare-except
    pattern: '(?m)^\s*except\s*:'
    message: "bare except catches everything — name the exception"
    severity: warning

  - id: compare-to-none
    pattern: '==\s*None\b'
    message: "use 'is None', not '== None'"
    severity: warning

  - id: assignment-in-condition
    pattern: '(?m)^\s*(?:if|elif|while)\s+[^#=]*[^=!<>]=[^=]'
    message: "single '=' assigns — did you mean '=='?"
    severity: error

  - id: mutable-default
    pattern: '(?m)^\s*def\s+(?<function>\w+)\s*\([^)]*=[ \t]*(?<default>\[\]|\{\})(?<rest>[^\n]*)'
    severity: error
    callback: |
      return function(c)
        if c.captures.rest:find("declint:allow") then
          return nil
        end
        return { message = "default " .. c.captures["default"] .. " in '" ..
                 c.captures["function"] .. "' is shared across calls" }
      end
    tests:
      - name: shared default is flagged
        text: "def f(a = []):"
        violations: 1
        messages: ["default [] in 'f' is shared across calls"]
      - name: allow marker vetoes
        text: "def f(a = []):  # declint:allow"
        violations: 0

scopes:
  # A top-level function — including `async def`. The region ends at the
  # next line that starts with any non-whitespace character (another
  # def/class, a decorator, or plain top-level code).
  - id: function
    start: '^(?:async\s+)?def\s'
    end: '^\S'
    rules:
      - id: print-in-function
        pattern: '\bprint\('
        message: "leftover print() — remove or use logging"
        severity: warning
        tests:
          - name: print inside a function is flagged
            text: "def f():\n    print(1)\n"
            violations: 1
          - name: top-level print is allowed
            text: "print(1)\n"
            violations: 0