Bump version to 0.0.73

Cargo.lock

@@ -1115,6 +1115,12 @@ version = "1.0.3"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "6c8af84674fe1f223a982c933a0ee1086ac4d4052aa0fb8060c12c6ad838e754"
 
+[[package]]
+name = "joinery"
+version = "2.1.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "72167d68f5fce3b8655487b8038691a3c9984ee769590f93f2a631f4ad64e4f5"
+
 [[package]]
 name = "js-sys"
 version = "0.3.60"
@@ -1960,7 +1966,7 @@ dependencies = [
 
 [[package]]
 name = "ruff"
-version = "0.0.69"
+version = "0.0.73"
 dependencies = [
  "anyhow",
  "assert_cmd",
@@ -1992,6 +1998,8 @@ dependencies = [
  "serde_json",
  "strum",
  "strum_macros",
+ "test-case",
+ "titlecase",
  "toml",
  "update-informer",
  "walkdir",
@@ -2405,6 +2413,28 @@ version = "0.2.4"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "507e9898683b6c43a9aa55b64259b721b52ba226e0f3779137e50ad114a4c90b"
 
+[[package]]
+name = "test-case"
+version = "2.2.2"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "21d6cf5a7dffb3f9dceec8e6b8ca528d9bd71d36c9f074defb548ce161f598c0"
+dependencies = [
+ "test-case-macros",
+]
+
+[[package]]
+name = "test-case-macros"
+version = "2.2.2"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "e45b7bf6e19353ddd832745c8fcf77a17a93171df7151187f26623f2b75b5b26"
+dependencies = [
+ "cfg-if 1.0.0",
+ "proc-macro-error",
+ "proc-macro2",
+ "quote",
+ "syn",
+]
+
 [[package]]
 name = "thiserror"
 version = "1.0.37"
@@ -2460,6 +2490,17 @@ version = "0.1.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "cda74da7e1a664f795bb1f8a87ec406fb89a02522cf6e50620d016add6dbbf5c"
 
+[[package]]
+name = "titlecase"
+version = "2.2.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "38397a8cdb017cfeb48bf6c154d6de975ac69ffeed35980fde199d2ee0842042"
+dependencies = [
+ "joinery",
+ "lazy_static",
+ "regex",
+]
+
 [[package]]
 name = "toml"
 version = "0.5.9"

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "ruff"
-version = "0.0.69"
+version = "0.0.73"
 edition = "2021"
 
 [lib]
@@ -23,25 +23,27 @@ itertools = { version = "0.10.5" }
 libcst = { git = "https://github.com/charliermarsh/LibCST", rev = "32a044c127668df44582f85699358e67803b0d73" }
 log = { version = "0.4.17" }
 notify = { version = "4.0.17" }
+num-bigint = { version = "0.4.3" }
 once_cell = { version = "1.13.1" }
 path-absolutize = { version = "3.0.13", features = ["once_cell_cache"] }
 rayon = { version = "1.5.3" }
 regex = { version = "1.6.0" }
 rustpython-ast = { features = ["unparse"], git = "https://github.com/charliermarsh/RustPython.git", rev = "778ae2aeb521d0438d2a91bd11238bb5c2bf9d4f" }
-rustpython-parser = { features = ["lalrpop"], git = "https://github.com/charliermarsh/RustPython.git", rev = "778ae2aeb521d0438d2a91bd11238bb5c2bf9d4f" }
 rustpython-common = { git = "https://github.com/charliermarsh/RustPython.git", rev = "778ae2aeb521d0438d2a91bd11238bb5c2bf9d4f" }
+rustpython-parser = { features = ["lalrpop"], git = "https://github.com/charliermarsh/RustPython.git", rev = "778ae2aeb521d0438d2a91bd11238bb5c2bf9d4f" }
 serde = { version = "1.0.143", features = ["derive"] }
 serde_json = { version = "1.0.83" }
+strum = { version = "0.24.1", features = ["strum_macros"] }
+strum_macros = { version = "0.24.3" }
+titlecase = { version = "2.2.1" }
 toml = { version = "0.5.9" }
 update-informer = { version = "0.5.0", default_features = false, features = ["pypi"], optional = true }
 walkdir = { version = "2.3.2" }
-strum = { version = "0.24.1", features = ["strum_macros"] }
-strum_macros = "0.24.3"
-num-bigint = "0.4.3"
 
 [dev-dependencies]
-assert_cmd = "2.0.4"
+assert_cmd = { version = "2.0.4" }
 insta = { version = "1.19.1", features = ["yaml"] }
+test-case = { version = "2.2.2" }
 
 [features]
 default = ["update-informer"]

README.md

@@ -57,7 +57,7 @@ ruff also works with [pre-commit](https://pre-commit.com):
 ```yaml
 repos:
   - repo: https://github.com/charliermarsh/ruff-pre-commit
-    rev: v0.0.48
+    rev: v0.0.73
     hooks:
       - id: lint
 ```
@@ -94,6 +94,8 @@ Arguments:
   <FILES>...
 
 Options:
+      --config <CONFIG>
+          Path to the `pyproject.toml` file to use for configuration
   -v, --verbose
           Enable verbose logging
   -q, --quiet
@@ -132,6 +134,8 @@ Options:
           Regular expression matching the name of dummy variables
       --target-version <TARGET_VERSION>
           The minimum Python version that should be supported
+      --stdin-filename <STDIN_FILENAME>
+          The name of the file when passing it through stdin
   -h, --help
           Print help information
   -V, --version
@@ -215,9 +219,9 @@ ruff also implements some of the most popular Flake8 plugins natively, including
 - [`flake8-builtins`](https://pypi.org/project/flake8-builtins/)
 - [`flake8-super`](https://pypi.org/project/flake8-super/)
 - [`flake8-print`](https://pypi.org/project/flake8-print/)
-- [`flake8-comprehensions`](https://pypi.org/project/flake8-comprehensions/) (11/16)
+- [`flake8-comprehensions`](https://pypi.org/project/flake8-comprehensions/) (15/16)
 - [`flake8-bugbear`](https://pypi.org/project/flake8-bugbear/) (3/32)
-- [`flake8-docstrings`](https://pypi.org/project/flake8-docstrings/) (11/47)
+- [`flake8-docstrings`](https://pypi.org/project/flake8-docstrings/) (37/48)
 - [`pyupgrade`](https://pypi.org/project/pyupgrade/) (8/34)
 
 Beyond rule-set parity, ruff suffers from the following limitations vis-à-vis Flake8:
@@ -293,7 +297,11 @@ The 🛠 emoji indicates that a rule is automatically fixable by the `--fix` com
 | C408 | UnnecessaryCollectionCall | Unnecessary <dict/list/tuple> call - rewrite as a literal |  |  |
 | C409 | UnnecessaryLiteralWithinTupleCall | Unnecessary <list/tuple> literal passed to tuple() - remove the outer call to tuple() |  |  |
 | C410 | UnnecessaryLiteralWithinListCall | Unnecessary <list/tuple> literal passed to list() - rewrite as a list literal |  |  |
+| C411 | UnnecessaryListCall | Unnecessary list call - remove the outer call to list() |  |  |
+| C413 | UnnecessaryCallAroundSorted | Unnecessary <list/reversed> call around sorted() |  |  |
+| C414 | UnnecessaryDoubleCastOrProcess | Unnecessary <list/reversed/set/sorted/tuple> call within <list/set/sorted/tuple>(). |  |  |
 | C415 | UnnecessarySubscriptReversal | Unnecessary subscript reversal of iterable within <reversed/set/sorted>() |  |  |
+| C416 | UnnecessaryComprehension |  Unnecessary <list/set> comprehension - rewrite using <list/set>() |  |  |
 | T201 | PrintFound | `print` found |  | 🛠 |
 | T203 | PPrintFound | `pprint` found |  | 🛠 |
 | U001 | UselessMetaclassType | `__metaclass__ = type` is implied |  | 🛠 |
@@ -304,16 +312,45 @@ The 🛠 emoji indicates that a rule is automatically fixable by the `--fix` com
 | U006 | UsePEP585Annotation | Use `list` instead of `List` for type annotations |  | 🛠 |
 | U007 | UsePEP604Annotation | Use `X \| Y` for type annotations |  | 🛠 |
 | U008 | SuperCallWithParameters | Use `super()` instead of `super(__class__, self)` |  | 🛠 |
+| D100 | PublicModule | Missing docstring in public module |  |  |
+| D101 | PublicClass | Missing docstring in public class |  |  |
+| D102 | PublicMethod | Missing docstring in public method |  |  |
+| D103 | PublicFunction | Missing docstring in public function |  |  |
+| D104 | PublicPackage | Missing docstring in public package |  |  |
+| D105 | MagicMethod | Missing docstring in magic method |  |  |
+| D106 | PublicNestedClass | Missing docstring in public nested class |  |  |
+| D107 | PublicInit | Missing docstring in __init__ |  |  |
 | D200 | FitsOnOneLine | One-line docstring should fit on one line |  |  |
+| D201 | NoBlankLineBeforeFunction | No blank lines allowed before function docstring (found 1) |  |  |
+| D202 | NoBlankLineAfterFunction | No blank lines allowed after function docstring (found 1) |  |  |
+| D203 | OneBlankLineBeforeClass | 1 blank line required before class docstring |  |  |
+| D204 | OneBlankLineAfterClass | 1 blank line required after class docstring |  |  |
 | D205 | NoBlankLineAfterSummary | 1 blank line required between summary line and description |  |  |
 | D209 | NewLineAfterLastParagraph | Multi-line docstring closing quotes should be on a separate line |  |  |
 | D210 | NoSurroundingWhitespace | No whitespaces allowed surrounding docstring text |  |  |
+| D211 | NoBlankLineBeforeClass | No blank lines allowed before class docstring |  |  |
 | D212 | MultiLineSummaryFirstLine | Multi-line docstring summary should start at the first line |  |  |
 | D213 | MultiLineSummarySecondLine | Multi-line docstring summary should start at the second line |  |  |
+| D214 | SectionNotOverIndented | Section is over-indented ("Returns") |  |  |
+| D215 | SectionUnderlineNotOverIndented | Section underline is over-indented ("Returns") |  |  |
 | D300 | UsesTripleQuotes | Use """triple double quotes""" |  |  |
 | D400 | EndsInPeriod | First line should end with a period |  |  |
+| D402 | NoSignature | First line should not be the function's 'signature' |  |  |
 | D403 | FirstLineCapitalized | First word of the first line should be properly capitalized |  |  |
+| D404 | NoThisPrefix | First word of the docstring should not be `This` |  |  |
+| D405 | CapitalizeSectionName | Section name should be properly capitalized ("returns") |  |  |
+| D406 | NewLineAfterSectionName | Section name should end with a newline ("Returns") |  |  |
+| D407 | DashedUnderlineAfterSection | Missing dashed underline after section ("Returns") |  |  |
+| D408 | SectionUnderlineAfterName | Section underline should be in the line following the section's name ("Returns") |  |  |
+| D409 | SectionUnderlineMatchesSectionLength | Section underline should match the length of its name ("Returns") |  |  |
+| D410 | BlankLineAfterSection | Missing blank line after section ("Returns") |  |  |
+| D411 | BlankLineBeforeSection | Missing blank line before section ("Returns") |  |  |
+| D412 | NoBlankLinesBetweenHeaderAndContent | No blank lines allowed between a section header and its content ("Returns") |  |  |
+| D413 | BlankLineAfterLastSection | Missing blank line after last section ("Returns") |  |  |
+| D414 | NonEmptySection | Section has no content ("Returns") |  |  |
 | D415 | EndsInPunctuation | First line should end with a period, question mark, or exclamation point |  |  |
+| D417 | DocumentAllArguments | Missing argument descriptions in the docstring: `x`, `y` |  |  |
+| D418 | SkipDocstring | Function decorated with @overload shouldn't contain a docstring |  |  |
 | D419 | NonEmpty | Docstring is empty |  |  |
 | M001 | UnusedNOQA | Unused `noqa` directive |  | 🛠 |
 

resources/test/fixtures/C411.py

@@ -0,0 +1,2 @@
+x = [1, 2, 3]
+list([i for i in x])

resources/test/fixtures/C413.py

@@ -0,0 +1,4 @@
+x = [2, 3, 1]
+list(sorted(x))
+reversed(sorted(x))
+reversed(sorted(x, reverse=True))

resources/test/fixtures/C414.py

@@ -0,0 +1,14 @@
+x = [1, 2, 3]
+list(list(x))
+list(tuple(x))
+tuple(list(x))
+tuple(tuple(x))
+set(set(x))
+set(list(x))
+set(tuple(x))
+set(sorted(x))
+set(reversed(x))
+sorted(list(x))
+sorted(tuple(x))
+sorted(sorted(x))
+sorted(reversed(x))

resources/test/fixtures/C416.py

@@ -0,0 +1,6 @@
+x = [1, 2, 3]
+[i for i in x]
+{i for i in x}
+
+[i for i in x if i > 1]
+[i for i in x for j in x]

resources/test/fixtures/canonical_numpy_examples.py

@@ -0,0 +1,163 @@
+"""This is the docstring for the example.py module.  Modules names should
+have short, all-lowercase names.  The module name may have underscores if
+this improves readability.
+
+Every module should have a docstring at the very top of the file.  The
+module's docstring may extend over multiple lines.  If your docstring does
+extend over multiple lines, the closing three quotation marks must be on
+a line by itself, preferably preceded by a blank line.
+
+"""
+
+# Example source file from the official "numpydoc docstring guide"
+# documentation (with the modification of commenting out all the original
+# ``import`` lines, plus adding this note and ``Expectation`` code):
+#     * As HTML: https://numpydoc.readthedocs.io/en/latest/example.html
+#     * Source Python:
+#         https://github.com/numpy/numpydoc/blob/master/doc/example.py
+
+# from __future__ import division, absolute_import, print_function
+#
+# import os  # standard library imports first
+#
+# Do NOT import using *, e.g. from numpy import *
+#
+# Import the module using
+#
+#   import numpy
+#
+# instead or import individual functions as needed, e.g
+#
+#  from numpy import array, zeros
+#
+# If you prefer the use of abbreviated module names, we suggest the
+# convention used by NumPy itself::
+#
+# import numpy as np
+# import matplotlib as mpl
+# import matplotlib.pyplot as plt
+#
+# These abbreviated names are not to be used in docstrings; users must
+# be able to paste and execute docstrings after importing only the
+# numpy module itself, unabbreviated.
+
+import os
+from .expected import Expectation
+
+expectation = Expectation()
+expect = expectation.expect
+
+# module docstring expected violations:
+expectation.expected.add((
+    os.path.normcase(__file__),
+    "D205: 1 blank line required between summary line and description "
+    "(found 0)"))
+expectation.expected.add((
+    os.path.normcase(__file__),
+    "D213: Multi-line docstring summary should start at the second line"))
+expectation.expected.add((
+    os.path.normcase(__file__),
+    "D400: First line should end with a period (not 'd')"))
+expectation.expected.add((
+    os.path.normcase(__file__),
+    "D404: First word of the docstring should not be `This`"))
+expectation.expected.add((
+    os.path.normcase(__file__),
+    "D415: First line should end with a period, question mark, or exclamation "
+    "point (not 'd')"))
+
+
+@expect("D213: Multi-line docstring summary should start at the second line",
+        arg_count=3)
+@expect("D401: First line should be in imperative mood; try rephrasing "
+        "(found 'A')", arg_count=3)
+@expect("D413: Missing blank line after last section ('Examples')",
+        arg_count=3)
+def foo(var1, var2, long_var_name='hi'):
+    r"""A one-line summary that does not use variable names.
+
+    Several sentences providing an extended description. Refer to
+    variables using back-ticks, e.g. `var`.
+
+    Parameters
+    ----------
+    var1 : array_like
+        Array_like means all those objects -- lists, nested lists, etc. --
+        that can be converted to an array.  We can also refer to
+        variables like `var1`.
+    var2 : int
+        The type above can either refer to an actual Python type
+        (e.g. ``int``), or describe the type of the variable in more
+        detail, e.g. ``(N,) ndarray`` or ``array_like``.
+    long_var_name : {'hi', 'ho'}, optional
+        Choices in brackets, default first when optional.
+
+    Returns
+    -------
+    type
+        Explanation of anonymous return value of type ``type``.
+    describe : type
+        Explanation of return value named `describe`.
+    out : type
+        Explanation of `out`.
+    type_without_description
+
+    Other Parameters
+    ----------------
+    only_seldom_used_keywords : type
+        Explanation
+    common_parameters_listed_above : type
+        Explanation
+
+    Raises
+    ------
+    BadException
+        Because you shouldn't have done that.
+
+    See Also
+    --------
+    numpy.array : Relationship (optional).
+    numpy.ndarray : Relationship (optional), which could be fairly long, in
+                    which case the line wraps here.
+    numpy.dot, numpy.linalg.norm, numpy.eye
+
+    Notes
+    -----
+    Notes about the implementation algorithm (if needed).
+
+    This can have multiple paragraphs.
+
+    You may include some math:
+
+    .. math:: X(e^{j\omega } ) = x(n)e^{ - j\omega n}
+
+    And even use a Greek symbol like :math:`\omega` inline.
+
+    References
+    ----------
+    Cite the relevant literature, e.g. [1]_.  You may also cite these
+    references in the notes section above.
+
+    .. [1] O. McNoleg, "The integration of GIS, remote sensing,
+       expert systems and adaptive co-kriging for environmental habitat
+       modelling of the Highland Haggis using object-oriented, fuzzy-logic
+       and neural-network techniques," Computers & Geosciences, vol. 22,
+       pp. 585-588, 1996.
+
+    Examples
+    --------
+    These are written in doctest format, and should illustrate how to
+    use the function.
+
+    >>> a = [1, 2, 3]
+    >>> print([x + 3 for x in a])
+    [4, 5, 6]
+    >>> print("a\nb")
+    a
+    b
+    """
+    # After closing class docstring, there should be one blank line to
+    # separate following codes (according to PEP257).
+    # But for function, method and module, there should be no blank lines
+    # after closing the docstring.
+    pass

resources/test/fixtures/sections.py

@@ -0,0 +1,497 @@
+"""A valid module docstring."""
+
+from .expected import Expectation
+
+expectation = Expectation()
+expect = expectation.expect
+
+
+_D213 = 'D213: Multi-line docstring summary should start at the second line'
+_D400 = "D400: First line should end with a period (not '!')"
+
+
+@expect(_D213)
+@expect("D405: Section name should be properly capitalized "
+        "('Returns', not 'returns')")
+def not_capitalized():  # noqa: D416
+    """Toggle the gizmo.
+
+    returns
+    -------
+    A value of some sort.
+
+    """
+
+
+@expect(_D213)
+@expect("D406: Section name should end with a newline "
+        "('Returns', not 'Returns:')")
+def superfluous_suffix():  # noqa: D416
+    """Toggle the gizmo.
+
+    Returns:
+    -------
+    A value of some sort.
+
+    """
+
+
+@expect(_D213)
+@expect("D407: Missing dashed underline after section ('Returns')")
+def no_underline():  # noqa: D416
+    """Toggle the gizmo.
+
+    Returns
+    A value of some sort.
+
+    """
+
+
+@expect(_D213)
+@expect("D407: Missing dashed underline after section ('Returns')")
+@expect("D414: Section has no content ('Returns')")
+def no_underline_and_no_description():  # noqa: D416
+    """Toggle the gizmo.
+
+    Returns
+
+    """
+
+
+@expect(_D213)
+@expect("D410: Missing blank line after section ('Returns')")
+@expect("D414: Section has no content ('Returns')")
+@expect("D411: Missing blank line before section ('Yields')")
+@expect("D414: Section has no content ('Yields')")
+def consecutive_sections():  # noqa: D416
+    """Toggle the gizmo.
+
+    Returns
+    -------
+    Yields
+    ------
+
+    Raises
+    ------
+    Questions.
+
+    """
+
+
+@expect(_D213)
+@expect("D408: Section underline should be in the line following the "
+        "section's name ('Returns')")
+def blank_line_before_underline():  # noqa: D416
+    """Toggle the gizmo.
+
+    Returns
+
+    -------
+    A value of some sort.
+
+    """
+
+
+@expect(_D213)
+@expect("D409: Section underline should match the length of its name "
+        "(Expected 7 dashes in section 'Returns', got 2)")
+def bad_underline_length():  # noqa: D416
+    """Toggle the gizmo.
+
+    Returns
+    --
+    A value of some sort.
+
+    """
+
+
+@expect(_D213)
+@expect("D413: Missing blank line after last section ('Returns')")
+def no_blank_line_after_last_section():  # noqa: D416
+    """Toggle the gizmo.
+
+    Returns
+    -------
+    A value of some sort.
+    """
+
+
+@expect(_D213)
+@expect("D411: Missing blank line before section ('Returns')")
+def no_blank_line_before_section():  # noqa: D416
+    """Toggle the gizmo.
+
+    The function's description.
+    Returns
+    -------
+    A value of some sort.
+
+    """
+
+
+@expect(_D213)
+@expect("D214: Section is over-indented ('Returns')")
+def section_overindented():  # noqa: D416
+    """Toggle the gizmo.
+
+        Returns
+    -------
+    A value of some sort.
+
+    """
+
+
+@expect(_D213)
+@expect("D215: Section underline is over-indented (in section 'Returns')")
+def section_underline_overindented():  # noqa: D416
+    """Toggle the gizmo.
+
+    Returns
+        -------
+    A value of some sort.
+
+    """
+
+
+@expect(_D213)
+@expect("D215: Section underline is over-indented (in section 'Returns')")
+@expect("D413: Missing blank line after last section ('Returns')")
+@expect("D414: Section has no content ('Returns')")
+def section_underline_overindented_and_contentless():  # noqa: D416
+    """Toggle the gizmo.
+
+    Returns
+        -------
+    """
+
+
+@expect(_D213)
+def ignore_non_actual_section():  # noqa: D416
+    """Toggle the gizmo.
+
+    This is the function's description, which will also specify what it
+    returns
+
+    """
+
+
+@expect(_D213)
+@expect("D401: First line should be in imperative mood "
+        "(perhaps 'Return', not 'Returns')")
+@expect("D400: First line should end with a period (not 's')")
+@expect("D415: First line should end with a period, question "
+        "mark, or exclamation point (not 's')")
+@expect("D205: 1 blank line required between summary line and description "
+        "(found 0)")
+def section_name_in_first_line():  # noqa: D416
+    """Returns
+    -------
+    A value of some sort.
+
+    """
+
+
+@expect(_D213)
+@expect("D405: Section name should be properly capitalized "
+        "('Short Summary', not 'Short summary')")
+@expect("D412: No blank lines allowed between a section header and its "
+        "content ('Short Summary')")
+@expect("D409: Section underline should match the length of its name "
+        "(Expected 7 dashes in section 'Returns', got 6)")
+@expect("D410: Missing blank line after section ('Returns')")
+@expect("D411: Missing blank line before section ('Raises')")
+@expect("D406: Section name should end with a newline "
+        "('Raises', not 'Raises:')")
+@expect("D407: Missing dashed underline after section ('Raises')")
+def multiple_sections():  # noqa: D416
+    """Toggle the gizmo.
+
+    Short summary
+    -------------
+
+    This is the function's description, which will also specify what it
+    returns.
+
+    Returns
+    ------
+    Many many wonderful things.
+    Raises:
+    My attention.
+
+    """
+
+
+@expect(_D213)
+def false_positive_section_prefix():  # noqa: D416
+    """Toggle the gizmo.
+
+    Parameters
+    ----------
+    attributes_are_fun: attributes for the function.
+
+    """
+
+
+@expect(_D213)
+def section_names_as_parameter_names():  # noqa: D416
+    """Toggle the gizmo.
+
+    Parameters
+    ----------
+    notes : list
+        A list of wonderful notes.
+    examples: list
+        A list of horrible examples.
+
+    """
+
+
+@expect(_D213)
+@expect("D414: Section has no content ('Returns')")
+def valid_google_style_section():  # noqa: D406, D407
+    """Toggle the gizmo.
+
+    Args:
+        note: A random string.
+
+    Returns:
+
+    Raises:
+        RandomError: A random error that occurs randomly.
+
+    """
+
+
+@expect(_D213)
+@expect("D416: Section name should end with a colon "
+        "('Args:', not 'Args')")
+def missing_colon_google_style_section():  # noqa: D406, D407
+    """Toggle the gizmo.
+
+    Args
+        note: A random string.
+
+    """
+
+
+@expect("D417: Missing argument descriptions in the docstring "
+        "(argument(s) y are missing descriptions in "
+        "'bar' docstring)", func_name="bar")
+def _test_nested_functions():
+    x = 1
+
+    def bar(y=2):  # noqa: D207, D213, D406, D407
+        """Nested function test for docstrings.
+
+        Will this work when referencing x?
+
+        Args:
+            x: Test something
+that is broken.
+
+        """
+        print(x)
+
+
+@expect(_D213)
+@expect("D417: Missing argument descriptions in the docstring "
+        "(argument(s) y are missing descriptions in "
+        "'test_missing_google_args' docstring)")
+def test_missing_google_args(x=1, y=2, _private=3):  # noqa: D406, D407
+    """Toggle the gizmo.
+
+    Args:
+        x (int): The greatest integer.
+
+    """
+
+
+class TestGoogle:  # noqa: D203
+    """Test class."""
+
+    def test_method(self, test, another_test, _):  # noqa: D213, D407
+        """Test a valid args section.
+
+        Args:
+            test: A parameter.
+            another_test: Another parameter.
+
+        """
+
+    @expect("D417: Missing argument descriptions in the docstring "
+            "(argument(s) test, y, z are missing descriptions in "
+            "'test_missing_args' docstring)", arg_count=5)
+    def test_missing_args(self, test, x, y, z=3, _private_arg=3):  # noqa: D213, D407
+        """Test a valid args section.
+
+        Args:
+            x: Another parameter.
+
+        """
+
+    @classmethod
+    @expect("D417: Missing argument descriptions in the docstring "
+            "(argument(s) test, y, z are missing descriptions in "
+            "'test_missing_args_class_method' docstring)", arg_count=5)
+    def test_missing_args_class_method(cls, test, x, y, _, z=3):  # noqa: D213, D407
+        """Test a valid args section.
+
+        Args:
+            x: Another parameter. The parameter below is missing description.
+            y:
+
+        """
+
+    @staticmethod
+    @expect("D417: Missing argument descriptions in the docstring "
+            "(argument(s) a, y, z are missing descriptions in "
+            "'test_missing_args_static_method' docstring)", arg_count=4)
+    def test_missing_args_static_method(a, x, y, _test, z=3):  # noqa: D213, D407
+        """Test a valid args section.
+
+        Args:
+            x: Another parameter.
+
+        """
+
+    @staticmethod
+    @expect("D417: Missing argument descriptions in the docstring "
+            "(argument(s) a, b are missing descriptions in "
+            "'test_missing_docstring' docstring)", arg_count=2)
+    def test_missing_docstring(a, b):  # noqa: D213, D407
+        """Test a valid args section.
+
+        Args:
+            a:
+
+        """
+
+    @staticmethod
+    def test_hanging_indent(skip, verbose):  # noqa: D213, D407
+        """Do stuff.
+
+        Args:
+            skip (:attr:`.Skip`):
+                Lorem ipsum dolor sit amet, consectetur adipiscing elit.
+                Etiam at tellus a tellus faucibus maximus. Curabitur tellus
+                mauris, semper id vehicula ac, feugiat ut tortor.
+            verbose (bool):
+                If True, print out as much infromation as possible.
+                If False, print out concise "one-liner" information.
+
+        """
+
+
+@expect(_D213)
+@expect("D417: Missing argument descriptions in the docstring "
+        "(argument(s) y are missing descriptions in "
+        "'test_missing_numpy_args' docstring)")
+def test_missing_numpy_args(_private_arg=0, x=1, y=2):  # noqa: D406, D407
+    """Toggle the gizmo.
+
+    Parameters
+    ----------
+    x : int
+        The greatest integer in the history \
+of the entire world.
+
+    """
+
+
+class TestNumpy:  # noqa: D203
+    """Test class."""
+
+    def test_method(self, test, another_test, z, _, x=1, y=2, _private_arg=1):  # noqa: D213, D407
+        """Test a valid args section.
+
+        Some long string with a \
+line continuation.
+
+        Parameters
+        ----------
+        test, another_test
+            Some parameters without type.
+        z : some parameter with a very long type description that requires a \
+line continuation.
+            But no further description.
+        x, y : int
+            Some integer parameters.
+
+        """
+
+    @expect("D417: Missing argument descriptions in the docstring "
+            "(argument(s) test, y, z are missing descriptions in "
+            "'test_missing_args' docstring)", arg_count=5)
+    def test_missing_args(self, test, x, y, z=3, t=1, _private=0):  # noqa: D213, D407
+        """Test a valid args section.
+
+        Parameters
+        ----------
+        x, t : int
+            Some parameters.
+
+
+        """
+
+    @classmethod
+    @expect("D417: Missing argument descriptions in the docstring "
+            "(argument(s) test, y, z are missing descriptions in "
+            "'test_missing_args_class_method' docstring)", arg_count=4)
+    def test_missing_args_class_method(cls, test, x, y, z=3):  # noqa: D213, D407
+        """Test a valid args section.
+
+        Parameters
+        ----------
+        z
+        x
+            Another parameter. The parameters y, test below are
+            missing descriptions. The parameter z above is also missing
+            a description.
+        y
+        test
+
+        """
+
+    @staticmethod
+    @expect("D417: Missing argument descriptions in the docstring "
+            "(argument(s) a, z are missing descriptions in "
+            "'test_missing_args_static_method' docstring)", arg_count=3)
+    def test_missing_args_static_method(a, x, y, z=3, t=1):  # noqa: D213, D407
+        """Test a valid args section.
+
+        Parameters
+        ----------
+        x, y
+            Another parameter.
+        t : int
+            Yet another parameter.
+
+        """
+
+    @staticmethod
+    def test_mixing_numpy_and_google(danger):  # noqa: D213
+        """Repro for #388.
+
+        Parameters
+        ----------
+        danger
+            Zoneeeeee!
+
+        """
+
+
+class TestIncorrectIndent:  # noqa: D203
+    """Test class."""
+
+    @expect("D417: Missing argument descriptions in the docstring "
+            "(argument(s) y are missing descriptions in "
+            "'test_incorrect_indent' docstring)", arg_count=3)
+    def test_incorrect_indent(self, x=1, y=2):  # noqa: D207, D213, D407
+        """Reproducing issue #437.
+
+Testing this incorrectly indented docstring.
+
+        Args:
+            x: Test argument.
+
+        """

src/ast/checkers.rs

@@ -4,8 +4,8 @@ use itertools::izip;
 use num_bigint::BigInt;
 use regex::Regex;
 use rustpython_parser::ast::{
-    Arg, ArgData, Arguments, Cmpop, Constant, Excepthandler, ExcepthandlerKind, Expr, ExprKind,
-    KeywordData, Located, Stmt, StmtKind, Unaryop,
+    Arg, ArgData, Arguments, Cmpop, Comprehension, Constant, Excepthandler, ExcepthandlerKind,
+    Expr, ExprKind, KeywordData, Located, Stmt, StmtKind, Unaryop,
 };
 use serde::{Deserialize, Serialize};
 
@@ -1004,6 +1004,104 @@ pub fn unnecessary_literal_within_list_call(
     None
 }
 
+pub fn unnecessary_list_call(expr: &Expr, func: &Expr, args: &[Expr]) -> Option<Check> {
+    if let ExprKind::Name { id, .. } = &func.node {
+        if id == "list" {
+            if let Some(arg) = args.first() {
+                if let ExprKind::ListComp { .. } = &arg.node {
+                    return Some(Check::new(
+                        CheckKind::UnnecessaryListCall,
+                        Range::from_located(expr),
+                    ));
+                }
+            }
+        }
+    }
+    None
+}
+
+pub fn unnecessary_call_around_sorted(expr: &Expr, func: &Expr, args: &[Expr]) -> Option<Check> {
+    if let ExprKind::Name { id: outer, .. } = &func.node {
+        if outer == "list" || outer == "reversed" {
+            if let Some(arg) = args.first() {
+                if let ExprKind::Call { func, .. } = &arg.node {
+                    if let ExprKind::Name { id: inner, .. } = &func.node {
+                        if inner == "sorted" {
+                            return Some(Check::new(
+                                CheckKind::UnnecessaryCallAroundSorted(outer.to_string()),
+                                Range::from_located(expr),
+                            ));
+                        }
+                    }
+                }
+            }
+        }
+    }
+    None
+}
+
+pub fn unnecessary_double_cast_or_process(
+    expr: &Expr,
+    func: &Expr,
+    args: &[Expr],
+) -> Option<Check> {
+    if let ExprKind::Name { id: outer, .. } = &func.node {
+        if outer == "list"
+            || outer == "tuple"
+            || outer == "set"
+            || outer == "reversed"
+            || outer == "sorted"
+        {
+            if let Some(arg) = args.first() {
+                if let ExprKind::Call { func, .. } = &arg.node {
+                    if let ExprKind::Name { id: inner, .. } = &func.node {
+                        // Ex) set(tuple(...))
+                        if (outer == "set" || outer == "sorted")
+                            && (inner == "list"
+                                || inner == "tuple"
+                                || inner == "reversed"
+                                || inner == "sorted")
+                        {
+                            return Some(Check::new(
+                                CheckKind::UnnecessaryDoubleCastOrProcess(
+                                    inner.to_string(),
+                                    outer.to_string(),
+                                ),
+                                Range::from_located(expr),
+                            ));
+                        }
+
+                        // Ex) list(tuple(...))
+                        if (outer == "list" || outer == "tuple")
+                            && (inner == "list" || inner == "tuple")
+                        {
+                            return Some(Check::new(
+                                CheckKind::UnnecessaryDoubleCastOrProcess(
+                                    inner.to_string(),
+                                    outer.to_string(),
+                                ),
+                                Range::from_located(expr),
+                            ));
+                        }
+
+                        // Ex) set(set(...))
+                        if outer == "set" && inner == "set" {
+                            return Some(Check::new(
+                                CheckKind::UnnecessaryDoubleCastOrProcess(
+                                    inner.to_string(),
+                                    outer.to_string(),
+                                ),
+                                Range::from_located(expr),
+                            ));
+                        }
+                    }
+                }
+            }
+        }
+    }
+    None
+}
+
 pub fn unnecessary_subscript_reversal(expr: &Expr, func: &Expr, args: &[Expr]) -> Option<Check> {
     if let Some(first_arg) = args.first() {
         if let ExprKind::Name { id, .. } = &func.node {
@@ -1042,6 +1140,41 @@ pub fn unnecessary_subscript_reversal(expr: &Expr, func: &Expr, args: &[Expr]) -
     None
 }
 
+pub fn unnecessary_comprehension(
+    expr: &Expr,
+    elt: &Located<ExprKind>,
+    generators: &Vec<Comprehension>,
+) -> Option<Check> {
+    if generators.len() == 1 {
+        let generator = &generators[0];
+        if generator.ifs.is_empty() && generator.is_async == 0 {
+            if let ExprKind::Name { id: elt_id, .. } = &elt.node {
+                if let ExprKind::Name { id: target_id, .. } = &generator.target.node {
+                    if elt_id == target_id {
+                        match &expr.node {
+                            ExprKind::ListComp { .. } => {
+                                return Some(Check::new(
+                                    CheckKind::UnnecessaryComprehension("list".to_string()),
+                                    Range::from_located(expr),
+                                ))
+                            }
+                            ExprKind::SetComp { .. } => {
+                                return Some(Check::new(
+                                    CheckKind::UnnecessaryComprehension("set".to_string()),
+                                    Range::from_located(expr),
+                                ))
+                            }
+                            _ => {}
+                        };
+                    }
+                }
+            }
+        }
+    }
+
+    None
+}
+
 // flake8-super
 /// Check that `super()` has no args
 pub fn super_args(

src/ast/operations.rs

@@ -134,7 +134,7 @@ impl<'a> SourceCodeLocator<'a> {
         }
     }
 
-    pub fn slice_source_code_at(&mut self, location: &Location) -> &'a str {
+    fn init(&mut self) {
         if !self.initialized {
             let mut offset = 0;
             for i in self.content.lines() {
@@ -142,24 +142,40 @@ impl<'a> SourceCodeLocator<'a> {
                 offset += i.len();
                 offset += 1;
             }
+            self.offsets.push(offset);
             self.initialized = true;
         }
+    }
+
+    pub fn slice_source_code_at(&mut self, location: &Location) -> &'a str {
+        self.init();
         let offset = self.offsets[location.row() - 1] + location.column() - 1;
         &self.content[offset..]
     }
 
     pub fn slice_source_code_range(&mut self, range: &Range) -> &'a str {
-        if !self.initialized {
-            let mut offset = 0;
-            for i in self.content.lines() {
-                self.offsets.push(offset);
-                offset += i.len();
-                offset += 1;
-            }
-            self.initialized = true;
-        }
+        self.init();
         let start = self.offsets[range.location.row() - 1] + range.location.column() - 1;
         let end = self.offsets[range.end_location.row() - 1] + range.end_location.column() - 1;
         &self.content[start..end]
     }
+
+    pub fn partition_source_code_at(
+        &mut self,
+        outer: &Range,
+        inner: &Range,
+    ) -> (&'a str, &'a str, &'a str) {
+        self.init();
+        let outer_start = self.offsets[outer.location.row() - 1] + outer.location.column() - 1;
+        let outer_end =
+            self.offsets[outer.end_location.row() - 1] + outer.end_location.column() - 1;
+        let inner_start = self.offsets[inner.location.row() - 1] + inner.location.column() - 1;
+        let inner_end =
+            self.offsets[inner.end_location.row() - 1] + inner.end_location.column() - 1;
+        (
+            &self.content[outer_start..inner_start],
+            &self.content[inner_start..inner_end],
+            &self.content[inner_end..outer_end],
+        )
+    }
 }

src/autofix/fixer.rs

@@ -1,7 +1,3 @@
-use std::fs;
-use std::path::Path;
-
-use anyhow::Result;
 use itertools::Itertools;
 use rustpython_parser::ast::Location;
 
@@ -24,17 +20,15 @@ impl From<bool> for Mode {
 }
 
 /// Auto-fix errors in a file, and write the fixed source code to disk.
-pub fn fix_file(checks: &mut [Check], contents: &str, path: &Path) -> Result<()> {
+pub fn fix_file(checks: &mut [Check], contents: &str) -> Option<String> {
     if checks.iter().all(|check| check.fix.is_none()) {
-        return Ok(());
+        return None;
     }
 
-    let output = apply_fixes(
+    Some(apply_fixes(
         checks.iter_mut().filter_map(|check| check.fix.as_mut()),
         contents,
-    );
-
-    fs::write(path, output).map_err(|e| e.into())
+    ))
 }
 
 /// Apply a series of fixes.

src/check_ast.rs

@@ -21,17 +21,19 @@ use crate::ast::visitor::{walk_excepthandler, Visitor};
 use crate::ast::{checkers, helpers, operations, visitor};
 use crate::autofix::{fixer, fixes};
 use crate::checks::{Check, CheckCode, CheckKind};
-use crate::docstrings::{Docstring, DocstringKind};
+use crate::docstrings::docstring_checks;
+use crate::docstrings::types::{Definition, DefinitionKind, Documentable};
+use crate::plugins;
 use crate::python::builtins::{BUILTINS, MAGIC_GLOBALS};
 use crate::python::future::ALL_FEATURE_NAMES;
 use crate::settings::{PythonVersion, Settings};
-use crate::{docstrings, plugins};
+use crate::visibility::{module_visibility, transition_scope, Modifier, Visibility, VisibleScope};
 
 pub const GLOBAL_SCOPE_INDEX: usize = 0;
 
 pub struct Checker<'a> {
     // Input data.
-    path: &'a Path,
+    pub(crate) path: &'a Path,
     // TODO(charlie): Separate immutable from mutable state. (None of these should ever change.)
     pub(crate) locator: SourceCodeLocator<'a>,
     pub(crate) settings: &'a Settings,
@@ -39,7 +41,7 @@ pub struct Checker<'a> {
     // Computed checks.
     checks: Vec<Check>,
     // Docstring tracking.
-    docstrings: Vec<Docstring<'a>>,
+    docstrings: Vec<(Definition<'a>, Visibility)>,
     // Edit tracking.
     // TODO(charlie): Instead of exposing deletions, wrap in a public API.
     pub(crate) deletions: BTreeSet<usize>,
@@ -52,11 +54,11 @@ pub struct Checker<'a> {
     dead_scopes: Vec<usize>,
     deferred_string_annotations: Vec<(Range, &'a str)>,
     deferred_annotations: Vec<(&'a Expr, Vec<usize>, Vec<usize>)>,
-    deferred_functions: Vec<(&'a Stmt, Vec<usize>, Vec<usize>)>,
+    deferred_functions: Vec<(&'a Stmt, Vec<usize>, Vec<usize>, VisibleScope)>,
     deferred_lambdas: Vec<(&'a Expr, Vec<usize>, Vec<usize>)>,
     deferred_assignments: Vec<usize>,
     // Internal, derivative state.
-    pub(crate) initial: bool,
+    visible_scope: VisibleScope,
     in_f_string: Option<Range>,
     in_annotation: bool,
     in_literal: bool,
@@ -91,7 +93,10 @@ impl<'a> Checker<'a> {
             deferred_functions: Default::default(),
             deferred_lambdas: Default::default(),
             deferred_assignments: Default::default(),
-            initial: true,
+            visible_scope: VisibleScope {
+                modifier: Modifier::Module,
+                visibility: module_visibility(path),
+            },
             in_f_string: None,
             in_annotation: Default::default(),
             in_literal: Default::default(),
@@ -125,36 +130,8 @@ where
             StmtKind::Import { .. } => {
                 self.futures_allowed = false;
             }
-            StmtKind::Expr { value } => {
-                // Track all docstrings: module-, class-, and function-level.
-                let mut is_module_docstring = false;
-                if matches!(
-                    &value.node,
-                    ExprKind::Constant {
-                        value: Constant::Str(_),
-                        ..
-                    }
-                ) {
-                    if let Some(docstring) = docstrings::extract(self, stmt, value) {
-                        if matches!(&docstring.kind, DocstringKind::Module) {
-                            is_module_docstring = true;
-                        }
-                        self.docstrings.push(docstring);
-                    }
-                }
-
-                if !is_module_docstring {
-                    if !self.seen_import_boundary
-                        && !operations::in_nested_block(&self.parent_stack, &self.parents)
-                    {
-                        self.seen_import_boundary = true;
-                    }
-                    self.futures_allowed = false;
-                }
-            }
             node => {
                 self.futures_allowed = false;
-
                 if !self.seen_import_boundary
                     && !helpers::is_assignment_to_a_dunder(node)
                     && !operations::in_nested_block(&self.parent_stack, &self.parents)
@@ -590,18 +567,39 @@ where
             StmtKind::Delete { .. } => {}
             _ => {}
         }
-        self.initial = false;
 
         // Recurse.
+        let prev_visibile_scope = self.visible_scope.clone();
         match &stmt.node {
-            StmtKind::FunctionDef { .. } | StmtKind::AsyncFunctionDef { .. } => {
+            StmtKind::FunctionDef { body, .. } | StmtKind::AsyncFunctionDef { body, .. } => {
+                let definition = docstring_checks::extract(
+                    &self.visible_scope,
+                    stmt,
+                    body,
+                    &Documentable::Function,
+                );
+                let scope = transition_scope(&self.visible_scope, stmt, &Documentable::Function);
+                self.docstrings.push((definition, scope.visibility.clone()));
+                self.visible_scope = scope;
+
                 self.deferred_functions.push((
                     stmt,
                     self.scope_stack.clone(),
                     self.parent_stack.clone(),
+                    self.visible_scope.clone(),
                 ));
             }
             StmtKind::ClassDef { body, .. } => {
+                let definition = docstring_checks::extract(
+                    &self.visible_scope,
+                    stmt,
+                    body,
+                    &Documentable::Class,
+                );
+                let scope = transition_scope(&self.visible_scope, stmt, &Documentable::Class);
+                self.docstrings.push((definition, scope.visibility.clone()));
+                self.visible_scope = scope;
+
                 for stmt in body {
                     self.visit_stmt(stmt);
                 }
@@ -629,6 +627,7 @@ where
             }
             _ => visitor::walk_stmt(self, stmt),
         };
+        self.visible_scope = prev_visibile_scope;
 
         // Post-visit.
         if let StmtKind::ClassDef { name, .. } = &stmt.node {
@@ -837,6 +836,28 @@ where
                         self.checks.push(check);
                     };
                 }
+
+                if self.settings.enabled.contains(&CheckCode::C411) {
+                    if let Some(check) = checkers::unnecessary_list_call(expr, func, args) {
+                        self.checks.push(check);
+                    };
+                }
+
+                if self.settings.enabled.contains(&CheckCode::C413) {
+                    if let Some(check) = checkers::unnecessary_call_around_sorted(expr, func, args)
+                    {
+                        self.checks.push(check);
+                    };
+                }
+
+                if self.settings.enabled.contains(&CheckCode::C414) {
+                    if let Some(check) =
+                        checkers::unnecessary_double_cast_or_process(expr, func, args)
+                    {
+                        self.checks.push(check);
+                    };
+                }
+
                 if self.settings.enabled.contains(&CheckCode::C415) {
                     if let Some(check) = checkers::unnecessary_subscript_reversal(expr, func, args)
                     {
@@ -1013,10 +1034,20 @@ where
                     self.visit_expr(expr);
                 }
             }
-            ExprKind::GeneratorExp { .. }
-            | ExprKind::ListComp { .. }
-            | ExprKind::DictComp { .. }
-            | ExprKind::SetComp { .. } => self.push_scope(Scope::new(ScopeKind::Generator)),
+
+            ExprKind::ListComp { elt, generators } | ExprKind::SetComp { elt, generators } => {
+                if self.settings.enabled.contains(&CheckCode::C416) {
+                    if let Some(check) = checkers::unnecessary_comprehension(expr, elt, generators)
+                    {
+                        self.checks.push(check);
+                    };
+                }
+                self.push_scope(Scope::new(ScopeKind::Generator))
+            }
+
+            ExprKind::GeneratorExp { .. } | ExprKind::DictComp { .. } => {
+                self.push_scope(Scope::new(ScopeKind::Generator))
+            }
             _ => {}
         };
 
@@ -1626,7 +1657,6 @@ impl<'a> Checker<'a> {
 
     fn handle_node_delete(&mut self, expr: &Expr) {
         if let ExprKind::Name { id, .. } = &expr.node {
-            // Check if we're on a conditional branch.
             if operations::on_conditional_branch(&self.parent_stack, &self.parents) {
                 return;
             }
@@ -1643,6 +1673,25 @@ impl<'a> Checker<'a> {
         }
     }
 
+    fn visit_docstring<'b>(&mut self, python_ast: &'b Suite) -> bool
+    where
+        'b: 'a,
+    {
+        let docstring = docstring_checks::docstring_from(python_ast);
+        self.docstrings.push((
+            Definition {
+                kind: if self.path.ends_with("__init__.py") {
+                    DefinitionKind::Package
+                } else {
+                    DefinitionKind::Module
+                },
+                docstring,
+            },
+            self.visible_scope.visibility.clone(),
+        ));
+        docstring.is_some()
+    }
+
     fn check_deferred_annotations(&mut self) {
         while let Some((expr, scopes, parents)) = self.deferred_annotations.pop() {
             self.parent_stack = parents;
@@ -1695,9 +1744,10 @@ impl<'a> Checker<'a> {
     }
 
     fn check_deferred_functions(&mut self) {
-        while let Some((stmt, scopes, parents)) = self.deferred_functions.pop() {
+        while let Some((stmt, scopes, parents, visibility)) = self.deferred_functions.pop() {
             self.parent_stack = parents;
             self.scope_stack = scopes;
+            self.visible_scope = visibility;
             self.push_scope(Scope::new(ScopeKind::Function(Default::default())));
 
             match &stmt.node {
@@ -1889,38 +1939,81 @@ impl<'a> Checker<'a> {
     }
 
     fn check_docstrings(&mut self) {
-        while let Some(docstring) = self.docstrings.pop() {
-            if !docstrings::not_empty(self, &docstring) {
+        while let Some((docstring, visibility)) = self.docstrings.pop() {
+            if !docstring_checks::not_empty(self, &docstring) {
+                continue;
+            }
+            if !docstring_checks::not_missing(self, &docstring, &visibility) {
                 continue;
             }
             if self.settings.enabled.contains(&CheckCode::D200) {
-                docstrings::one_liner(self, &docstring);
+                docstring_checks::one_liner(self, &docstring);
+            }
+            if self.settings.enabled.contains(&CheckCode::D201)
+                || self.settings.enabled.contains(&CheckCode::D202)
+            {
+                docstring_checks::blank_before_after_function(self, &docstring);
+            }
+            if self.settings.enabled.contains(&CheckCode::D203)
+                || self.settings.enabled.contains(&CheckCode::D204)
+                || self.settings.enabled.contains(&CheckCode::D211)
+            {
+                docstring_checks::blank_before_after_class(self, &docstring);
             }
             if self.settings.enabled.contains(&CheckCode::D205) {
-                docstrings::blank_after_summary(self, &docstring);
+                docstring_checks::blank_after_summary(self, &docstring);
             }
             if self.settings.enabled.contains(&CheckCode::D209) {
-                docstrings::newline_after_last_paragraph(self, &docstring);
+                docstring_checks::newline_after_last_paragraph(self, &docstring);
             }
             if self.settings.enabled.contains(&CheckCode::D210) {
-                docstrings::no_surrounding_whitespace(self, &docstring);
+                docstring_checks::no_surrounding_whitespace(self, &docstring);
             }
             if self.settings.enabled.contains(&CheckCode::D212)
                 || self.settings.enabled.contains(&CheckCode::D213)
             {
-                docstrings::multi_line_summary_start(self, &docstring);
+                docstring_checks::multi_line_summary_start(self, &docstring);
             }
             if self.settings.enabled.contains(&CheckCode::D300) {
-                docstrings::triple_quotes(self, &docstring);
+                docstring_checks::triple_quotes(self, &docstring);
             }
             if self.settings.enabled.contains(&CheckCode::D400) {
-                docstrings::ends_with_period(self, &docstring);
+                docstring_checks::ends_with_period(self, &docstring);
+            }
+            if self.settings.enabled.contains(&CheckCode::D402) {
+                docstring_checks::no_signature(self, &docstring);
             }
             if self.settings.enabled.contains(&CheckCode::D403) {
-                docstrings::capitalized(self, &docstring);
+                docstring_checks::capitalized(self, &docstring);
+            }
+            if self.settings.enabled.contains(&CheckCode::D404) {
+                docstring_checks::starts_with_this(self, &docstring);
             }
             if self.settings.enabled.contains(&CheckCode::D415) {
-                docstrings::ends_with_punctuation(self, &docstring);
+                docstring_checks::ends_with_punctuation(self, &docstring);
+            }
+            if self.settings.enabled.contains(&CheckCode::D418) {
+                docstring_checks::if_needed(self, &docstring);
+            }
+            if self.settings.enabled.contains(&CheckCode::D212)
+                || self.settings.enabled.contains(&CheckCode::D214)
+                || self.settings.enabled.contains(&CheckCode::D215)
+                || self.settings.enabled.contains(&CheckCode::D405)
+                || self.settings.enabled.contains(&CheckCode::D406)
+                || self.settings.enabled.contains(&CheckCode::D407)
+                || self.settings.enabled.contains(&CheckCode::D407)
+                || self.settings.enabled.contains(&CheckCode::D408)
+                || self.settings.enabled.contains(&CheckCode::D409)
+                || self.settings.enabled.contains(&CheckCode::D410)
+                || self.settings.enabled.contains(&CheckCode::D411)
+                || self.settings.enabled.contains(&CheckCode::D412)
+                || self.settings.enabled.contains(&CheckCode::D413)
+                || self.settings.enabled.contains(&CheckCode::D414)
+                || self.settings.enabled.contains(&CheckCode::D414)
+                || self.settings.enabled.contains(&CheckCode::D414)
+                || self.settings.enabled.contains(&CheckCode::D417)
+            {
+                docstring_checks::check_sections(self, &docstring);
             }
         }
     }
@@ -1974,6 +2067,13 @@ pub fn check_ast(
     checker.push_scope(Scope::new(ScopeKind::Module));
     checker.bind_builtins();
 
+    // Check for module docstring.
+    let python_ast = if checker.visit_docstring(python_ast) {
+        &python_ast[1..]
+    } else {
+        python_ast
+    };
+
     // Iterate over the AST.
     for stmt in python_ast {
         checker.visit_stmt(stmt);

src/checks.rs

@@ -137,7 +137,11 @@ pub enum CheckCode {
     C408,
     C409,
     C410,
+    C411,
+    C413,
+    C414,
     C415,
+    C416,
     // flake8-print
     T201,
     T203,
@@ -151,16 +155,45 @@ pub enum CheckCode {
     U007,
     U008,
     // pydocstyle
+    D100,
+    D101,
+    D102,
+    D103,
+    D104,
+    D105,
+    D106,
+    D107,
     D200,
+    D201,
+    D202,
+    D203,
+    D204,
     D205,
     D209,
     D210,
+    D211,
     D212,
     D213,
+    D214,
+    D215,
     D300,
     D400,
+    D402,
     D403,
+    D404,
+    D405,
+    D406,
+    D407,
+    D408,
+    D409,
+    D410,
+    D411,
+    D412,
+    D413,
+    D414,
     D415,
+    D417,
+    D418,
     D419,
     // Meta
     M001,
@@ -245,7 +278,11 @@ pub enum CheckKind {
     UnnecessaryCollectionCall(String),
     UnnecessaryLiteralWithinTupleCall(String),
     UnnecessaryLiteralWithinListCall(String),
+    UnnecessaryListCall,
+    UnnecessaryCallAroundSorted(String),
+    UnnecessaryDoubleCastOrProcess(String, String),
     UnnecessarySubscriptReversal(String),
+    UnnecessaryComprehension(String),
     // flake8-print
     PrintFound,
     PPrintFound,
@@ -259,16 +296,45 @@ pub enum CheckKind {
     UsePEP604Annotation,
     SuperCallWithParameters,
     // pydocstyle
+    BlankLineAfterLastSection(String),
+    BlankLineAfterSection(String),
+    BlankLineBeforeSection(String),
+    CapitalizeSectionName(String),
+    DashedUnderlineAfterSection(String),
+    DocumentAllArguments(Vec<String>),
     EndsInPeriod,
     EndsInPunctuation,
     FirstLineCapitalized,
     FitsOnOneLine,
+    MagicMethod,
     MultiLineSummaryFirstLine,
     MultiLineSummarySecondLine,
     NewLineAfterLastParagraph,
+    NewLineAfterSectionName(String),
+    NoBlankLineAfterFunction(usize),
     NoBlankLineAfterSummary,
+    NoBlankLineBeforeClass(usize),
+    NoBlankLineBeforeFunction(usize),
+    NoBlankLinesBetweenHeaderAndContent(String),
+    NoSignature,
     NoSurroundingWhitespace,
+    NoThisPrefix,
     NonEmpty,
+    NonEmptySection(String),
+    OneBlankLineAfterClass(usize),
+    OneBlankLineBeforeClass(usize),
+    PublicClass,
+    PublicFunction,
+    PublicInit,
+    PublicMethod,
+    PublicModule,
+    PublicNestedClass,
+    PublicPackage,
+    SectionNotOverIndented(String),
+    SectionUnderlineAfterName(String),
+    SectionUnderlineMatchesSectionLength(String),
+    SectionUnderlineNotOverIndented(String),
+    SkipDocstring,
     UsesTripleQuotes,
     // Meta
     UnusedNOQA(Option<Vec<String>>),
@@ -360,9 +426,18 @@ impl CheckCode {
             CheckCode::C410 => {
                 CheckKind::UnnecessaryLiteralWithinListCall("<list/tuple>".to_string())
             }
+            CheckCode::C411 => CheckKind::UnnecessaryListCall,
+            CheckCode::C413 => {
+                CheckKind::UnnecessaryCallAroundSorted("<list/reversed>".to_string())
+            }
+            CheckCode::C414 => CheckKind::UnnecessaryDoubleCastOrProcess(
+                "<list/reversed/set/sorted/tuple>".to_string(),
+                "<list/set/sorted/tuple>".to_string(),
+            ),
             CheckCode::C415 => {
                 CheckKind::UnnecessarySubscriptReversal("<reversed/set/sorted>".to_string())
             }
+            CheckCode::C416 => CheckKind::UnnecessaryComprehension("<list/set>".to_string()),
             // flake8-print
             CheckCode::T201 => CheckKind::PrintFound,
             CheckCode::T203 => CheckKind::PPrintFound,
@@ -379,17 +454,52 @@ impl CheckCode {
             CheckCode::U007 => CheckKind::UsePEP604Annotation,
             CheckCode::U008 => CheckKind::SuperCallWithParameters,
             // pydocstyle
+            CheckCode::D100 => CheckKind::PublicModule,
+            CheckCode::D101 => CheckKind::PublicClass,
+            CheckCode::D102 => CheckKind::PublicMethod,
+            CheckCode::D103 => CheckKind::PublicFunction,
+            CheckCode::D104 => CheckKind::PublicPackage,
+            CheckCode::D105 => CheckKind::MagicMethod,
+            CheckCode::D106 => CheckKind::PublicNestedClass,
+            CheckCode::D107 => CheckKind::PublicInit,
             CheckCode::D200 => CheckKind::FitsOnOneLine,
+            CheckCode::D201 => CheckKind::NoBlankLineBeforeFunction(1),
+            CheckCode::D202 => CheckKind::NoBlankLineAfterFunction(1),
+            CheckCode::D203 => CheckKind::OneBlankLineBeforeClass(0),
+            CheckCode::D204 => CheckKind::OneBlankLineAfterClass(0),
             CheckCode::D205 => CheckKind::NoBlankLineAfterSummary,
             CheckCode::D209 => CheckKind::NewLineAfterLastParagraph,
             CheckCode::D210 => CheckKind::NoSurroundingWhitespace,
-            CheckCode::D400 => CheckKind::EndsInPeriod,
-            CheckCode::D419 => CheckKind::NonEmpty,
+            CheckCode::D211 => CheckKind::NoBlankLineBeforeClass(1),
             CheckCode::D212 => CheckKind::MultiLineSummaryFirstLine,
             CheckCode::D213 => CheckKind::MultiLineSummarySecondLine,
             CheckCode::D300 => CheckKind::UsesTripleQuotes,
+            CheckCode::D400 => CheckKind::EndsInPeriod,
+            CheckCode::D402 => CheckKind::NoSignature,
             CheckCode::D403 => CheckKind::FirstLineCapitalized,
+            CheckCode::D404 => CheckKind::NoThisPrefix,
+            CheckCode::D405 => CheckKind::CapitalizeSectionName("returns".to_string()),
+            CheckCode::D406 => CheckKind::NewLineAfterSectionName("Returns".to_string()),
+            CheckCode::D407 => CheckKind::DashedUnderlineAfterSection("Returns".to_string()),
+            CheckCode::D408 => CheckKind::SectionUnderlineAfterName("Returns".to_string()),
+            CheckCode::D409 => {
+                CheckKind::SectionUnderlineMatchesSectionLength("Returns".to_string())
+            }
+            CheckCode::D410 => CheckKind::BlankLineAfterSection("Returns".to_string()),
+            CheckCode::D411 => CheckKind::BlankLineBeforeSection("Returns".to_string()),
+            CheckCode::D412 => {
+                CheckKind::NoBlankLinesBetweenHeaderAndContent("Returns".to_string())
+            }
+            CheckCode::D413 => CheckKind::BlankLineAfterLastSection("Returns".to_string()),
+            CheckCode::D414 => CheckKind::NonEmptySection("Returns".to_string()),
             CheckCode::D415 => CheckKind::EndsInPunctuation,
+            CheckCode::D418 => CheckKind::SkipDocstring,
+            CheckCode::D419 => CheckKind::NonEmpty,
+            CheckCode::D214 => CheckKind::SectionNotOverIndented("Returns".to_string()),
+            CheckCode::D215 => CheckKind::SectionUnderlineNotOverIndented("Returns".to_string()),
+            CheckCode::D417 => {
+                CheckKind::DocumentAllArguments(vec!["x".to_string(), "y".to_string()])
+            }
             // Meta
             CheckCode::M001 => CheckKind::UnusedNOQA(None),
         }
@@ -464,7 +574,11 @@ impl CheckKind {
             CheckKind::UnnecessaryCollectionCall(_) => &CheckCode::C408,
             CheckKind::UnnecessaryLiteralWithinTupleCall(..) => &CheckCode::C409,
             CheckKind::UnnecessaryLiteralWithinListCall(..) => &CheckCode::C410,
+            CheckKind::UnnecessaryListCall => &CheckCode::C411,
+            CheckKind::UnnecessaryCallAroundSorted(_) => &CheckCode::C413,
+            CheckKind::UnnecessaryDoubleCastOrProcess(..) => &CheckCode::C414,
             CheckKind::UnnecessarySubscriptReversal(_) => &CheckCode::C415,
+            CheckKind::UnnecessaryComprehension(..) => &CheckCode::C416,
             // flake8-print
             CheckKind::PrintFound => &CheckCode::T201,
             CheckKind::PPrintFound => &CheckCode::T203,
@@ -478,17 +592,46 @@ impl CheckKind {
             CheckKind::UselessObjectInheritance(_) => &CheckCode::U004,
             CheckKind::SuperCallWithParameters => &CheckCode::U008,
             // pydocstyle
-            CheckKind::FitsOnOneLine => &CheckCode::D200,
-            CheckKind::NoBlankLineAfterSummary => &CheckCode::D205,
-            CheckKind::NewLineAfterLastParagraph => &CheckCode::D209,
-            CheckKind::NoSurroundingWhitespace => &CheckCode::D210,
+            CheckKind::BlankLineAfterLastSection(_) => &CheckCode::D413,
+            CheckKind::BlankLineAfterSection(_) => &CheckCode::D410,
+            CheckKind::BlankLineBeforeSection(_) => &CheckCode::D411,
+            CheckKind::CapitalizeSectionName(_) => &CheckCode::D405,
+            CheckKind::DashedUnderlineAfterSection(_) => &CheckCode::D407,
+            CheckKind::DocumentAllArguments(_) => &CheckCode::D417,
             CheckKind::EndsInPeriod => &CheckCode::D400,
-            CheckKind::NonEmpty => &CheckCode::D419,
+            CheckKind::EndsInPunctuation => &CheckCode::D415,
+            CheckKind::FirstLineCapitalized => &CheckCode::D403,
+            CheckKind::FitsOnOneLine => &CheckCode::D200,
+            CheckKind::MagicMethod => &CheckCode::D105,
             CheckKind::MultiLineSummaryFirstLine => &CheckCode::D212,
             CheckKind::MultiLineSummarySecondLine => &CheckCode::D213,
+            CheckKind::NewLineAfterLastParagraph => &CheckCode::D209,
+            CheckKind::NewLineAfterSectionName(_) => &CheckCode::D406,
+            CheckKind::NoBlankLineAfterFunction(_) => &CheckCode::D202,
+            CheckKind::NoBlankLineAfterSummary => &CheckCode::D205,
+            CheckKind::NoBlankLineBeforeClass(_) => &CheckCode::D211,
+            CheckKind::NoBlankLineBeforeFunction(_) => &CheckCode::D201,
+            CheckKind::NoBlankLinesBetweenHeaderAndContent(_) => &CheckCode::D412,
+            CheckKind::NoSignature => &CheckCode::D402,
+            CheckKind::NoSurroundingWhitespace => &CheckCode::D210,
+            CheckKind::NoThisPrefix => &CheckCode::D404,
+            CheckKind::NonEmpty => &CheckCode::D419,
+            CheckKind::NonEmptySection(_) => &CheckCode::D414,
+            CheckKind::OneBlankLineAfterClass(_) => &CheckCode::D204,
+            CheckKind::OneBlankLineBeforeClass(_) => &CheckCode::D203,
+            CheckKind::PublicClass => &CheckCode::D101,
+            CheckKind::PublicFunction => &CheckCode::D103,
+            CheckKind::PublicInit => &CheckCode::D107,
+            CheckKind::PublicMethod => &CheckCode::D102,
+            CheckKind::PublicModule => &CheckCode::D100,
+            CheckKind::PublicNestedClass => &CheckCode::D106,
+            CheckKind::PublicPackage => &CheckCode::D104,
+            CheckKind::SectionNotOverIndented(_) => &CheckCode::D214,
+            CheckKind::SectionUnderlineAfterName(_) => &CheckCode::D408,
+            CheckKind::SectionUnderlineMatchesSectionLength(_) => &CheckCode::D409,
+            CheckKind::SectionUnderlineNotOverIndented(_) => &CheckCode::D215,
+            CheckKind::SkipDocstring => &CheckCode::D418,
             CheckKind::UsesTripleQuotes => &CheckCode::D300,
-            CheckKind::FirstLineCapitalized => &CheckCode::D403,
-            CheckKind::EndsInPunctuation => &CheckCode::D415,
             // Meta
             CheckKind::UnusedNOQA(_) => &CheckCode::M001,
         }
@@ -701,9 +844,21 @@ impl CheckKind {
                     )
                 }
             }
+            CheckKind::UnnecessaryListCall => {
+                "Unnecessary list call - remove the outer call to list()".to_string()
+            }
+            CheckKind::UnnecessaryCallAroundSorted(func) => {
+                format!("Unnecessary {func} call around sorted()")
+            }
+            CheckKind::UnnecessaryDoubleCastOrProcess(inner, outer) => {
+                format!("Unnecessary {inner} call within {outer}().")
+            }
             CheckKind::UnnecessarySubscriptReversal(func) => {
                 format!("Unnecessary subscript reversal of iterable within {func}()")
             }
+            CheckKind::UnnecessaryComprehension(obj_type) => {
+                format!(" Unnecessary {obj_type} comprehension - rewrite using {obj_type}()")
+            }
             // flake8-print
             CheckKind::PrintFound => "`print` found".to_string(),
             CheckKind::PPrintFound => "`pprint` found".to_string(),
@@ -759,6 +914,83 @@ impl CheckKind {
             CheckKind::MultiLineSummarySecondLine => {
                 "Multi-line docstring summary should start at the second line".to_string()
             }
+            CheckKind::NoSignature => {
+                "First line should not be the function's 'signature'".to_string()
+            }
+            CheckKind::NoBlankLineBeforeFunction(num_lines) => {
+                format!("No blank lines allowed before function docstring (found {num_lines})")
+            }
+            CheckKind::NoBlankLineAfterFunction(num_lines) => {
+                format!("No blank lines allowed after function docstring (found {num_lines})")
+            }
+            CheckKind::NoBlankLineBeforeClass(_) => {
+                "No blank lines allowed before class docstring".to_string()
+            }
+            CheckKind::OneBlankLineBeforeClass(_) => {
+                "1 blank line required before class docstring".to_string()
+            }
+            CheckKind::OneBlankLineAfterClass(_) => {
+                "1 blank line required after class docstring".to_string()
+            }
+            CheckKind::PublicModule => "Missing docstring in public module".to_string(),
+            CheckKind::PublicClass => "Missing docstring in public class".to_string(),
+            CheckKind::PublicMethod => "Missing docstring in public method".to_string(),
+            CheckKind::PublicFunction => "Missing docstring in public function".to_string(),
+            CheckKind::PublicPackage => "Missing docstring in public package".to_string(),
+            CheckKind::MagicMethod => "Missing docstring in magic method".to_string(),
+            CheckKind::PublicNestedClass => "Missing docstring in public nested class".to_string(),
+            CheckKind::PublicInit => "Missing docstring in __init__".to_string(),
+            CheckKind::NoThisPrefix => {
+                "First word of the docstring should not be `This`".to_string()
+            }
+            CheckKind::SkipDocstring => {
+                "Function decorated with @overload shouldn't contain a docstring".to_string()
+            }
+            CheckKind::CapitalizeSectionName(name) => {
+                format!("Section name should be properly capitalized (\"{name}\")")
+            }
+            CheckKind::BlankLineAfterLastSection(name) => {
+                format!("Missing blank line after last section (\"{name}\")")
+            }
+            CheckKind::BlankLineAfterSection(name) => {
+                format!("Missing blank line after section (\"{name}\")")
+            }
+            CheckKind::BlankLineBeforeSection(name) => {
+                format!("Missing blank line before section (\"{name}\")")
+            }
+            CheckKind::NewLineAfterSectionName(name) => {
+                format!("Section name should end with a newline (\"{name}\")")
+            }
+            CheckKind::DashedUnderlineAfterSection(name) => {
+                format!("Missing dashed underline after section (\"{name}\")")
+            }
+            CheckKind::SectionUnderlineAfterName(name) => {
+                format!("Section underline should be in the line following the section's name (\"{name}\")")
+            }
+            CheckKind::SectionUnderlineMatchesSectionLength(name) => {
+                format!("Section underline should match the length of its name (\"{name}\")")
+            }
+            CheckKind::NoBlankLinesBetweenHeaderAndContent(name) => {
+                format!(
+                    "No blank lines allowed between a section header and its content (\"{name}\")"
+                )
+            }
+            CheckKind::NonEmptySection(name) => format!("Section has no content (\"{name}\")"),
+            CheckKind::SectionNotOverIndented(name) => {
+                format!("Section is over-indented (\"{name}\")")
+            }
+            CheckKind::SectionUnderlineNotOverIndented(name) => {
+                format!("Section underline is over-indented (\"{name}\")")
+            }
+            CheckKind::DocumentAllArguments(names) => {
+                if names.len() == 1 {
+                    let name = &names[0];
+                    format!("Missing argument description in the docstring: `{name}`")
+                } else {
+                    let names = names.iter().map(|name| format!("`{name}`")).join(", ");
+                    format!("Missing argument descriptions in the docstring: {names}")
+                }
+            }
             // Meta
             CheckKind::UnusedNOQA(codes) => match codes {
                 None => "Unused `noqa` directive".to_string(),

src/cli.rs

@@ -17,6 +17,9 @@ use crate::RawSettings;
 pub struct Cli {
     #[arg(required = true)]
     pub files: Vec<PathBuf>,
+    /// Path to the `pyproject.toml` file to use for configuration.
+    #[arg(long)]
+    pub config: Option<PathBuf>,
     /// Enable verbose logging.
     #[arg(short, long)]
     pub verbose: bool,

src/docstrings.rs

@@ -1,325 +1,3 @@
-use rustpython_ast::{Constant, Expr, ExprKind, Location, Stmt, StmtKind};
-
-use crate::ast::types::Range;
-use crate::check_ast::Checker;
-use crate::checks::{Check, CheckCode, CheckKind};
-
-#[derive(Debug)]
-pub enum DocstringKind {
-    Module,
-    Function,
-    Class,
-}
-
-#[derive(Debug)]
-pub struct Docstring<'a> {
-    pub kind: DocstringKind,
-    pub parent: Option<&'a Stmt>,
-    pub expr: &'a Expr,
-}
-
-/// Extract a `Docstring` from an `Expr`.
-pub fn extract<'a, 'b>(
-    checker: &'a Checker,
-    stmt: &'b Stmt,
-    expr: &'b Expr,
-) -> Option<Docstring<'b>> {
-    let defined_in = checker
-        .binding_context()
-        .defined_in
-        .map(|index| checker.parents[index]);
-    match defined_in {
-        None => {
-            if checker.initial {
-                return Some(Docstring {
-                    kind: DocstringKind::Module,
-                    parent: None,
-                    expr,
-                });
-            }
-        }
-        Some(parent) => {
-            if let StmtKind::FunctionDef { body, .. }
-            | StmtKind::AsyncFunctionDef { body, .. }
-            | StmtKind::ClassDef { body, .. } = &parent.node
-            {
-                if body.first().map(|node| node == stmt).unwrap_or_default() {
-                    return Some(Docstring {
-                        kind: if matches!(&parent.node, StmtKind::ClassDef { .. }) {
-                            DocstringKind::Class
-                        } else {
-                            DocstringKind::Function
-                        },
-                        parent: None,
-                        expr,
-                    });
-                }
-            }
-        }
-    }
-
-    None
-}
-
-/// Extract the source code range for a `Docstring`.
-fn range_for(docstring: &Docstring) -> Range {
-    // RustPython currently omits the first quotation mark in a string, so offset the location.
-    Range {
-        location: Location::new(
-            docstring.expr.location.row(),
-            docstring.expr.location.column() - 1,
-        ),
-        end_location: docstring.expr.end_location,
-    }
-}
-
-/// D200
-pub fn one_liner(checker: &mut Checker, docstring: &Docstring) {
-    if let ExprKind::Constant {
-        value: Constant::Str(string),
-        ..
-    } = &docstring.expr.node
-    {
-        let mut line_count = 0;
-        let mut non_empty_line_count = 0;
-        for line in string.lines() {
-            line_count += 1;
-            if !line.trim().is_empty() {
-                non_empty_line_count += 1;
-            }
-            if non_empty_line_count > 1 {
-                return;
-            }
-        }
-
-        if non_empty_line_count == 1 && line_count > 1 {
-            checker.add_check(Check::new(CheckKind::FitsOnOneLine, range_for(docstring)));
-        }
-    }
-}
-
-/// D205
-pub fn blank_after_summary(checker: &mut Checker, docstring: &Docstring) {
-    if let ExprKind::Constant {
-        value: Constant::Str(string),
-        ..
-    } = &docstring.expr.node
-    {
-        let mut lines_count = 1;
-        let mut blanks_count = 0;
-        for line in string.trim().lines().skip(1) {
-            lines_count += 1;
-            if line.trim().is_empty() {
-                blanks_count += 1;
-            } else {
-                break;
-            }
-        }
-        if lines_count > 1 && blanks_count != 1 {
-            checker.add_check(Check::new(
-                CheckKind::NoBlankLineAfterSummary,
-                range_for(docstring),
-            ));
-        }
-    }
-}
-
-/// D209
-pub fn newline_after_last_paragraph(checker: &mut Checker, docstring: &Docstring) {
-    if let ExprKind::Constant {
-        value: Constant::Str(string),
-        ..
-    } = &docstring.expr.node
-    {
-        let mut line_count = 0;
-        for line in string.lines() {
-            if !line.trim().is_empty() {
-                line_count += 1;
-            }
-            if line_count > 1 {
-                let content = checker
-                    .locator
-                    .slice_source_code_range(&range_for(docstring));
-                if let Some(line) = content.lines().last() {
-                    let line = line.trim();
-                    if line != "\"\"\"" && line != "'''" {
-                        checker.add_check(Check::new(
-                            CheckKind::NewLineAfterLastParagraph,
-                            range_for(docstring),
-                        ));
-                    }
-                }
-                return;
-            }
-        }
-    }
-}
-
-/// D210
-pub fn no_surrounding_whitespace(checker: &mut Checker, docstring: &Docstring) {
-    if let ExprKind::Constant {
-        value: Constant::Str(string),
-        ..
-    } = &docstring.expr.node
-    {
-        let mut lines = string.lines();
-        if let Some(line) = lines.next() {
-            if line.trim().is_empty() {
-                return;
-            }
-            if line.starts_with(' ') || (matches!(lines.next(), None) && line.ends_with(' ')) {
-                checker.add_check(Check::new(
-                    CheckKind::NoSurroundingWhitespace,
-                    range_for(docstring),
-                ));
-            }
-        }
-    }
-}
-
-/// D212, D213
-pub fn multi_line_summary_start(checker: &mut Checker, docstring: &Docstring) {
-    if let ExprKind::Constant {
-        value: Constant::Str(string),
-        ..
-    } = &docstring.expr.node
-    {
-        if string.lines().nth(1).is_some() {
-            let content = checker
-                .locator
-                .slice_source_code_range(&range_for(docstring));
-            if let Some(first_line) = content.lines().next() {
-                let first_line = first_line.trim();
-                if first_line == "\"\"\""
-                    || first_line == "'''"
-                    || first_line == "u\"\"\""
-                    || first_line == "u'''"
-                    || first_line == "r\"\"\""
-                    || first_line == "r'''"
-                    || first_line == "ur\"\"\""
-                    || first_line == "ur'''"
-                {
-                    if checker.settings.enabled.contains(&CheckCode::D212) {
-                        checker.add_check(Check::new(
-                            CheckKind::MultiLineSummaryFirstLine,
-                            range_for(docstring),
-                        ));
-                    }
-                } else if checker.settings.enabled.contains(&CheckCode::D213) {
-                    checker.add_check(Check::new(
-                        CheckKind::MultiLineSummarySecondLine,
-                        range_for(docstring),
-                    ));
-                }
-            }
-        }
-    }
-}
-
-/// D300
-pub fn triple_quotes(checker: &mut Checker, docstring: &Docstring) {
-    if let ExprKind::Constant {
-        value: Constant::Str(string),
-        ..
-    } = &docstring.expr.node
-    {
-        let content = checker
-            .locator
-            .slice_source_code_range(&range_for(docstring));
-        if string.contains("\"\"\"") {
-            if !content.starts_with("'''") {
-                checker.add_check(Check::new(
-                    CheckKind::UsesTripleQuotes,
-                    range_for(docstring),
-                ));
-            }
-        } else if !content.starts_with("\"\"\"") {
-            checker.add_check(Check::new(
-                CheckKind::UsesTripleQuotes,
-                range_for(docstring),
-            ));
-        }
-    }
-}
-
-/// D400
-pub fn ends_with_period(checker: &mut Checker, docstring: &Docstring) {
-    if let ExprKind::Constant {
-        value: Constant::Str(string),
-        ..
-    } = &docstring.expr.node
-    {
-        if let Some(string) = string.lines().next() {
-            if !string.ends_with('.') {
-                checker.add_check(Check::new(CheckKind::EndsInPeriod, range_for(docstring)));
-            }
-        }
-    }
-}
-
-/// D403
-pub fn capitalized(checker: &mut Checker, docstring: &Docstring) {
-    if !matches!(docstring.kind, DocstringKind::Function) {
-        return;
-    }
-
-    if let ExprKind::Constant {
-        value: Constant::Str(string),
-        ..
-    } = &docstring.expr.node
-    {
-        if let Some(first_word) = string.split(' ').next() {
-            if first_word == first_word.to_uppercase() {
-                return;
-            }
-            for char in first_word.chars() {
-                if !char.is_ascii_alphabetic() && char != '\'' {
-                    return;
-                }
-            }
-            if let Some(first_char) = first_word.chars().next() {
-                if !first_char.is_uppercase() {
-                    checker.add_check(Check::new(
-                        CheckKind::FirstLineCapitalized,
-                        range_for(docstring),
-                    ));
-                }
-            }
-        }
-    }
-}
-
-/// D415
-pub fn ends_with_punctuation(checker: &mut Checker, docstring: &Docstring) {
-    if let ExprKind::Constant {
-        value: Constant::Str(string),
-        ..
-    } = &docstring.expr.node
-    {
-        if let Some(string) = string.lines().next() {
-            if !(string.ends_with('.') || string.ends_with('!') || string.ends_with('?')) {
-                checker.add_check(Check::new(
-                    CheckKind::EndsInPunctuation,
-                    range_for(docstring),
-                ));
-            }
-        }
-    }
-}
-
-/// D419
-pub fn not_empty(checker: &mut Checker, docstring: &Docstring) -> bool {
-    if let ExprKind::Constant {
-        value: Constant::Str(string),
-        ..
-    } = &docstring.expr.node
-    {
-        if string.trim().is_empty() {
-            if checker.settings.enabled.contains(&CheckCode::D419) {
-                checker.add_check(Check::new(CheckKind::NonEmpty, range_for(docstring)));
-            }
-            return false;
-        }
-    }
-    true
-}
+pub mod docstring_checks;
+pub mod sections;
+pub mod types;

src/docstrings/docstring_checks.rs

@@ -0,0 +1,676 @@
+//! Abstractions for tracking and validating docstrings in Python code.
+
+use once_cell::sync::Lazy;
+use regex::Regex;
+use rustpython_ast::{Constant, Expr, ExprKind, Location, Stmt, StmtKind};
+
+use crate::ast::types::Range;
+use crate::check_ast::Checker;
+use crate::checks::{Check, CheckCode, CheckKind};
+use crate::docstrings::sections::{check_numpy_section, section_contexts};
+use crate::docstrings::types::{Definition, DefinitionKind, Documentable};
+use crate::visibility::{is_init, is_magic, is_overload, Modifier, Visibility, VisibleScope};
+
+/// Extract a docstring from a function or class body.
+pub fn docstring_from(suite: &[Stmt]) -> Option<&Expr> {
+    if let Some(stmt) = suite.first() {
+        if let StmtKind::Expr { value } = &stmt.node {
+            if matches!(
+                &value.node,
+                ExprKind::Constant {
+                    value: Constant::Str(_),
+                    ..
+                }
+            ) {
+                return Some(value);
+            }
+        }
+    }
+    None
+}
+
+/// Extract a `Definition` from the AST node defined by a `Stmt`.
+pub fn extract<'a>(
+    scope: &VisibleScope,
+    stmt: &'a Stmt,
+    body: &'a [Stmt],
+    kind: &Documentable,
+) -> Definition<'a> {
+    let expr = docstring_from(body);
+    match kind {
+        Documentable::Function => match scope {
+            VisibleScope {
+                modifier: Modifier::Module,
+                ..
+            } => Definition {
+                kind: DefinitionKind::Function(stmt),
+                docstring: expr,
+            },
+            VisibleScope {
+                modifier: Modifier::Class,
+                ..
+            } => Definition {
+                kind: DefinitionKind::Method(stmt),
+                docstring: expr,
+            },
+            VisibleScope {
+                modifier: Modifier::Function,
+                ..
+            } => Definition {
+                kind: DefinitionKind::NestedFunction(stmt),
+                docstring: expr,
+            },
+        },
+        Documentable::Class => match scope {
+            VisibleScope {
+                modifier: Modifier::Module,
+                ..
+            } => Definition {
+                kind: DefinitionKind::Class(stmt),
+                docstring: expr,
+            },
+            VisibleScope {
+                modifier: Modifier::Class,
+                ..
+            } => Definition {
+                kind: DefinitionKind::NestedClass(stmt),
+                docstring: expr,
+            },
+            VisibleScope {
+                modifier: Modifier::Function,
+                ..
+            } => Definition {
+                kind: DefinitionKind::NestedClass(stmt),
+                docstring: expr,
+            },
+        },
+    }
+}
+
+/// Extract the source code range for a docstring.
+pub fn range_for(docstring: &Expr) -> Range {
+    // RustPython currently omits the first quotation mark in a string, so offset the location.
+    Range {
+        location: Location::new(docstring.location.row(), docstring.location.column() - 1),
+        end_location: docstring.end_location,
+    }
+}
+
+/// D100, D101, D102, D103, D104, D105, D106, D107
+pub fn not_missing(
+    checker: &mut Checker,
+    definition: &Definition,
+    visibility: &Visibility,
+) -> bool {
+    if matches!(visibility, Visibility::Private) {
+        return true;
+    }
+
+    if definition.docstring.is_some() {
+        return true;
+    }
+
+    match definition.kind {
+        DefinitionKind::Module => {
+            if checker.settings.enabled.contains(&CheckCode::D100) {
+                checker.add_check(Check::new(
+                    CheckKind::PublicModule,
+                    Range {
+                        location: Location::new(1, 1),
+                        end_location: Location::new(1, 1),
+                    },
+                ));
+            }
+            false
+        }
+        DefinitionKind::Package => {
+            if checker.settings.enabled.contains(&CheckCode::D104) {
+                checker.add_check(Check::new(
+                    CheckKind::PublicPackage,
+                    Range {
+                        location: Location::new(1, 1),
+                        end_location: Location::new(1, 1),
+                    },
+                ));
+            }
+            false
+        }
+        DefinitionKind::Class(stmt) => {
+            if checker.settings.enabled.contains(&CheckCode::D101) {
+                checker.add_check(Check::new(
+                    CheckKind::PublicClass,
+                    Range::from_located(stmt),
+                ));
+            }
+            false
+        }
+        DefinitionKind::NestedClass(stmt) => {
+            if checker.settings.enabled.contains(&CheckCode::D106) {
+                checker.add_check(Check::new(
+                    CheckKind::PublicNestedClass,
+                    Range::from_located(stmt),
+                ));
+            }
+            false
+        }
+        DefinitionKind::Function(stmt) | DefinitionKind::NestedFunction(stmt) => {
+            if is_overload(stmt) {
+                true
+            } else {
+                if checker.settings.enabled.contains(&CheckCode::D103) {
+                    checker.add_check(Check::new(
+                        CheckKind::PublicFunction,
+                        Range::from_located(stmt),
+                    ));
+                }
+                false
+            }
+        }
+        DefinitionKind::Method(stmt) => {
+            if is_overload(stmt) {
+                true
+            } else if is_magic(stmt) {
+                if checker.settings.enabled.contains(&CheckCode::D105) {
+                    checker.add_check(Check::new(
+                        CheckKind::MagicMethod,
+                        Range::from_located(stmt),
+                    ));
+                }
+                true
+            } else if is_init(stmt) {
+                if checker.settings.enabled.contains(&CheckCode::D107) {
+                    checker.add_check(Check::new(CheckKind::PublicInit, Range::from_located(stmt)));
+                }
+                true
+            } else {
+                if checker.settings.enabled.contains(&CheckCode::D102) {
+                    checker.add_check(Check::new(
+                        CheckKind::PublicMethod,
+                        Range::from_located(stmt),
+                    ));
+                }
+                true
+            }
+        }
+    }
+}
+
+/// D200
+pub fn one_liner(checker: &mut Checker, definition: &Definition) {
+    if let Some(docstring) = &definition.docstring {
+        if let ExprKind::Constant {
+            value: Constant::Str(string),
+            ..
+        } = &docstring.node
+        {
+            let mut line_count = 0;
+            let mut non_empty_line_count = 0;
+            for line in string.lines() {
+                line_count += 1;
+                if !line.trim().is_empty() {
+                    non_empty_line_count += 1;
+                }
+                if non_empty_line_count > 1 {
+                    break;
+                }
+            }
+
+            if non_empty_line_count == 1 && line_count > 1 {
+                checker.add_check(Check::new(CheckKind::FitsOnOneLine, range_for(docstring)));
+            }
+        }
+    }
+}
+
+static COMMENT_REGEX: Lazy<Regex> = Lazy::new(|| Regex::new(r"^\s*#").unwrap());
+
+static INNER_FUNCTION_OR_CLASS_REGEX: Lazy<Regex> =
+    Lazy::new(|| Regex::new(r"^\s+(?:(?:class|def|async def)\s|@)").unwrap());
+
+/// D201, D202
+pub fn blank_before_after_function(checker: &mut Checker, definition: &Definition) {
+    if let Some(docstring) = definition.docstring {
+        if let DefinitionKind::Function(parent)
+        | DefinitionKind::NestedFunction(parent)
+        | DefinitionKind::Method(parent) = &definition.kind
+        {
+            if let ExprKind::Constant {
+                value: Constant::Str(_),
+                ..
+            } = &docstring.node
+            {
+                let (before, _, after) = checker
+                    .locator
+                    .partition_source_code_at(&Range::from_located(parent), &range_for(docstring));
+
+                if checker.settings.enabled.contains(&CheckCode::D201) {
+                    let blank_lines_before = before
+                        .lines()
+                        .rev()
+                        .skip(1)
+                        .take_while(|line| line.trim().is_empty())
+                        .count();
+                    if blank_lines_before != 0 {
+                        checker.add_check(Check::new(
+                            CheckKind::NoBlankLineBeforeFunction(blank_lines_before),
+                            range_for(docstring),
+                        ));
+                    }
+                }
+
+                if checker.settings.enabled.contains(&CheckCode::D202) {
+                    let blank_lines_after = after
+                        .lines()
+                        .skip(1)
+                        .take_while(|line| line.trim().is_empty())
+                        .count();
+                    let all_blank_after = after
+                        .lines()
+                        .skip(1)
+                        .all(|line| line.trim().is_empty() || COMMENT_REGEX.is_match(line));
+                    // Report a D202 violation if the docstring is followed by a blank line
+                    // and the blank line is not itself followed by an inner function or
+                    // class.
+                    if !all_blank_after
+                        && blank_lines_after != 0
+                        && !(blank_lines_after == 1
+                            && INNER_FUNCTION_OR_CLASS_REGEX.is_match(after))
+                    {
+                        checker.add_check(Check::new(
+                            CheckKind::NoBlankLineAfterFunction(blank_lines_after),
+                            range_for(docstring),
+                        ));
+                    }
+                }
+            }
+        }
+    }
+}
+
+/// D203, D204, D211
+pub fn blank_before_after_class(checker: &mut Checker, definition: &Definition) {
+    if let Some(docstring) = &definition.docstring {
+        if let DefinitionKind::Class(parent) | DefinitionKind::NestedClass(parent) =
+            &definition.kind
+        {
+            if let ExprKind::Constant {
+                value: Constant::Str(_),
+                ..
+            } = &docstring.node
+            {
+                let (before, _, after) = checker
+                    .locator
+                    .partition_source_code_at(&Range::from_located(parent), &range_for(docstring));
+
+                if checker.settings.enabled.contains(&CheckCode::D203)
+                    || checker.settings.enabled.contains(&CheckCode::D211)
+                {
+                    let blank_lines_before = before
+                        .lines()
+                        .rev()
+                        .skip(1)
+                        .take_while(|line| line.trim().is_empty())
+                        .count();
+                    if blank_lines_before != 0
+                        && checker.settings.enabled.contains(&CheckCode::D211)
+                    {
+                        checker.add_check(Check::new(
+                            CheckKind::NoBlankLineBeforeClass(blank_lines_before),
+                            range_for(docstring),
+                        ));
+                    }
+                    if blank_lines_before != 1
+                        && checker.settings.enabled.contains(&CheckCode::D203)
+                    {
+                        checker.add_check(Check::new(
+                            CheckKind::OneBlankLineBeforeClass(blank_lines_before),
+                            range_for(docstring),
+                        ));
+                    }
+                }
+
+                if checker.settings.enabled.contains(&CheckCode::D204) {
+                    let blank_lines_after = after
+                        .lines()
+                        .skip(1)
+                        .take_while(|line| line.trim().is_empty())
+                        .count();
+                    let all_blank_after = after
+                        .lines()
+                        .skip(1)
+                        .all(|line| line.trim().is_empty() || COMMENT_REGEX.is_match(line));
+                    if !all_blank_after && blank_lines_after != 1 {
+                        checker.add_check(Check::new(
+                            CheckKind::OneBlankLineAfterClass(blank_lines_after),
+                            range_for(docstring),
+                        ));
+                    }
+                }
+            }
+        }
+    }
+}
+
+/// D205
+pub fn blank_after_summary(checker: &mut Checker, definition: &Definition) {
+    if let Some(docstring) = definition.docstring {
+        if let ExprKind::Constant {
+            value: Constant::Str(string),
+            ..
+        } = &docstring.node
+        {
+            let mut lines_count = 1;
+            let mut blanks_count = 0;
+            for line in string.trim().lines().skip(1) {
+                lines_count += 1;
+                if line.trim().is_empty() {
+                    blanks_count += 1;
+                } else {
+                    break;
+                }
+            }
+            if lines_count > 1 && blanks_count != 1 {
+                checker.add_check(Check::new(
+                    CheckKind::NoBlankLineAfterSummary,
+                    range_for(docstring),
+                ));
+            }
+        }
+    }
+}
+
+/// D209
+pub fn newline_after_last_paragraph(checker: &mut Checker, definition: &Definition) {
+    if let Some(docstring) = definition.docstring {
+        if let ExprKind::Constant {
+            value: Constant::Str(string),
+            ..
+        } = &docstring.node
+        {
+            let mut line_count = 0;
+            for line in string.lines() {
+                if !line.trim().is_empty() {
+                    line_count += 1;
+                }
+                if line_count > 1 {
+                    let content = checker
+                        .locator
+                        .slice_source_code_range(&range_for(docstring));
+                    if let Some(line) = content.lines().last() {
+                        let line = line.trim();
+                        if line != "\"\"\"" && line != "'''" {
+                            checker.add_check(Check::new(
+                                CheckKind::NewLineAfterLastParagraph,
+                                range_for(docstring),
+                            ));
+                        }
+                    }
+                    return;
+                }
+            }
+        }
+    }
+}
+
+/// D210
+pub fn no_surrounding_whitespace(checker: &mut Checker, definition: &Definition) {
+    if let Some(docstring) = definition.docstring {
+        if let ExprKind::Constant {
+            value: Constant::Str(string),
+            ..
+        } = &docstring.node
+        {
+            let mut lines = string.lines();
+            if let Some(line) = lines.next() {
+                if line.trim().is_empty() {
+                    return;
+                }
+                if line.starts_with(' ') || (matches!(lines.next(), None) && line.ends_with(' ')) {
+                    checker.add_check(Check::new(
+                        CheckKind::NoSurroundingWhitespace,
+                        range_for(docstring),
+                    ));
+                }
+            }
+        }
+    }
+}
+
+/// D212, D213
+pub fn multi_line_summary_start(checker: &mut Checker, definition: &Definition) {
+    if let Some(docstring) = definition.docstring {
+        if let ExprKind::Constant {
+            value: Constant::Str(string),
+            ..
+        } = &docstring.node
+        {
+            if string.lines().nth(1).is_some() {
+                let content = checker
+                    .locator
+                    .slice_source_code_range(&range_for(docstring));
+                if let Some(first_line) = content.lines().next() {
+                    let first_line = first_line.trim();
+                    if first_line == "\"\"\"" || first_line == "'''" {
+                        if checker.settings.enabled.contains(&CheckCode::D212) {
+                            checker.add_check(Check::new(
+                                CheckKind::MultiLineSummaryFirstLine,
+                                range_for(docstring),
+                            ));
+                        }
+                    } else if checker.settings.enabled.contains(&CheckCode::D213) {
+                        checker.add_check(Check::new(
+                            CheckKind::MultiLineSummarySecondLine,
+                            range_for(docstring),
+                        ));
+                    }
+                }
+            }
+        }
+    }
+}
+
+/// D300
+pub fn triple_quotes(checker: &mut Checker, definition: &Definition) {
+    if let Some(docstring) = definition.docstring {
+        if let ExprKind::Constant {
+            value: Constant::Str(string),
+            ..
+        } = &docstring.node
+        {
+            let content = checker
+                .locator
+                .slice_source_code_range(&range_for(docstring));
+            if string.contains("\"\"\"") {
+                if !content.starts_with("'''") {
+                    checker.add_check(Check::new(
+                        CheckKind::UsesTripleQuotes,
+                        range_for(docstring),
+                    ));
+                }
+            } else if !content.starts_with("\"\"\"") {
+                checker.add_check(Check::new(
+                    CheckKind::UsesTripleQuotes,
+                    range_for(docstring),
+                ));
+            }
+        }
+    }
+}
+
+/// D400
+pub fn ends_with_period(checker: &mut Checker, definition: &Definition) {
+    if let Some(docstring) = definition.docstring {
+        if let ExprKind::Constant {
+            value: Constant::Str(string),
+            ..
+        } = &docstring.node
+        {
+            if let Some(string) = string.lines().next() {
+                if !string.ends_with('.') {
+                    checker.add_check(Check::new(CheckKind::EndsInPeriod, range_for(docstring)));
+                }
+            }
+        }
+    }
+}
+
+/// D402
+pub fn no_signature(checker: &mut Checker, definition: &Definition) {
+    if let Some(docstring) = definition.docstring {
+        if let DefinitionKind::Function(parent)
+        | DefinitionKind::NestedFunction(parent)
+        | DefinitionKind::Method(parent) = definition.kind
+        {
+            if let StmtKind::FunctionDef { name, .. } = &parent.node {
+                if let ExprKind::Constant {
+                    value: Constant::Str(string),
+                    ..
+                } = &docstring.node
+                {
+                    if let Some(first_line) = string.lines().next() {
+                        if first_line.contains(&format!("{name}(")) {
+                            checker.add_check(Check::new(
+                                CheckKind::NoSignature,
+                                range_for(docstring),
+                            ));
+                        }
+                    }
+                }
+            }
+        }
+    }
+}
+
+/// D403
+pub fn capitalized(checker: &mut Checker, definition: &Definition) {
+    if !matches!(definition.kind, DefinitionKind::Function(_)) {
+        return;
+    }
+
+    if let Some(docstring) = definition.docstring {
+        if let ExprKind::Constant {
+            value: Constant::Str(string),
+            ..
+        } = &docstring.node
+        {
+            if let Some(first_word) = string.split(' ').next() {
+                if first_word == first_word.to_uppercase() {
+                    return;
+                }
+                for char in first_word.chars() {
+                    if !char.is_ascii_alphabetic() && char != '\'' {
+                        return;
+                    }
+                }
+                if let Some(first_char) = first_word.chars().next() {
+                    if !first_char.is_uppercase() {
+                        checker.add_check(Check::new(
+                            CheckKind::FirstLineCapitalized,
+                            range_for(docstring),
+                        ));
+                    }
+                }
+            }
+        }
+    }
+}
+
+/// D404
+pub fn starts_with_this(checker: &mut Checker, definition: &Definition) {
+    if let Some(docstring) = definition.docstring {
+        if let ExprKind::Constant {
+            value: Constant::Str(string),
+            ..
+        } = &docstring.node
+        {
+            let trimmed = string.trim();
+            if trimmed.is_empty() {
+                return;
+            }
+
+            if let Some(first_word) = string.split(' ').next() {
+                if first_word
+                    .replace(|c: char| !c.is_alphanumeric(), "")
+                    .to_lowercase()
+                    == "this"
+                {
+                    checker.add_check(Check::new(CheckKind::NoThisPrefix, range_for(docstring)));
+                }
+            }
+        }
+    }
+}
+
+/// D415
+pub fn ends_with_punctuation(checker: &mut Checker, definition: &Definition) {
+    if let Some(docstring) = definition.docstring {
+        if let ExprKind::Constant {
+            value: Constant::Str(string),
+            ..
+        } = &docstring.node
+        {
+            if let Some(string) = string.lines().next() {
+                if !(string.ends_with('.') || string.ends_with('!') || string.ends_with('?')) {
+                    checker.add_check(Check::new(
+                        CheckKind::EndsInPunctuation,
+                        range_for(docstring),
+                    ));
+                }
+            }
+        }
+    }
+}
+
+/// D418
+pub fn if_needed(checker: &mut Checker, definition: &Definition) {
+    if definition.docstring.is_some() {
+        if let DefinitionKind::Function(stmt)
+        | DefinitionKind::NestedFunction(stmt)
+        | DefinitionKind::Method(stmt) = definition.kind
+        {
+            if is_overload(stmt) {
+                checker.add_check(Check::new(
+                    CheckKind::SkipDocstring,
+                    Range::from_located(stmt),
+                ));
+            }
+        }
+    }
+}
+
+/// D419
+pub fn not_empty(checker: &mut Checker, definition: &Definition) -> bool {
+    if let Some(docstring) = definition.docstring {
+        if let ExprKind::Constant {
+            value: Constant::Str(string),
+            ..
+        } = &docstring.node
+        {
+            if string.trim().is_empty() {
+                if checker.settings.enabled.contains(&CheckCode::D419) {
+                    checker.add_check(Check::new(CheckKind::NonEmpty, range_for(docstring)));
+                }
+                return false;
+            }
+        }
+    }
+    true
+}
+
+pub fn check_sections(checker: &mut Checker, definition: &Definition) {
+    if let Some(docstring) = definition.docstring {
+        if let ExprKind::Constant {
+            value: Constant::Str(string),
+            ..
+        } = &docstring.node
+        {
+            let lines: Vec<&str> = string.lines().collect();
+            if lines.len() < 2 {
+                return;
+            }
+            for context in &section_contexts(&lines) {
+                check_numpy_section(checker, definition, context);
+            }
+        }
+    }
+}

src/docstrings/sections.rs

@@ -0,0 +1,512 @@
+use itertools::Itertools;
+use std::collections::BTreeSet;
+
+use once_cell::sync::Lazy;
+use rustpython_ast::{Arg, Expr, Location, StmtKind};
+use titlecase::titlecase;
+
+use crate::ast::types::Range;
+use crate::check_ast::Checker;
+use crate::checks::{Check, CheckCode, CheckKind};
+use crate::docstrings::docstring_checks::range_for;
+use crate::docstrings::types::{Definition, DefinitionKind};
+use crate::visibility::is_static;
+
+static NUMPY_SECTION_NAMES: Lazy<BTreeSet<&'static str>> = Lazy::new(|| {
+    BTreeSet::from([
+        "Short Summary",
+        "Extended Summary",
+        "Parameters",
+        "Returns",
+        "Yields",
+        "Other Parameters",
+        "Raises",
+        "See Also",
+        "Notes",
+        "References",
+        "Examples",
+        "Attributes",
+        "Methods",
+    ])
+});
+
+static NUMPY_SECTION_NAMES_LOWERCASE: Lazy<BTreeSet<&'static str>> = Lazy::new(|| {
+    BTreeSet::from([
+        "short summary",
+        "extended summary",
+        "parameters",
+        "returns",
+        "yields",
+        "other parameters",
+        "raises",
+        "see also",
+        "notes",
+        "references",
+        "examples",
+        "attributes",
+        "methods",
+    ])
+});
+
+// TODO(charlie): Include Google section names.
+// static GOOGLE_SECTION_NAMES: Lazy<BTreeSet<&'static str>> = Lazy::new(|| {
+//     BTreeSet::from([
+//         "Args",
+//         "Arguments",
+//         "Attention",
+//         "Attributes",
+//         "Caution",
+//         "Danger",
+//         "Error",
+//         "Example",
+//         "Examples",
+//         "Hint",
+//         "Important",
+//         "Keyword Args",
+//         "Keyword Arguments",
+//         "Methods",
+//         "Note",
+//         "Notes",
+//         "Return",
+//         "Returns",
+//         "Raises",
+//         "References",
+//         "See Also",
+//         "Tip",
+//         "Todo",
+//         "Warning",
+//         "Warnings",
+//         "Warns",
+//         "Yield",
+//         "Yields",
+//     ])
+// });
+
+fn indentation<'a>(checker: &'a mut Checker, docstring: &Expr) -> &'a str {
+    let range = range_for(docstring);
+    checker.locator.slice_source_code_range(&Range {
+        location: Location::new(range.location.row(), 1),
+        end_location: Location::new(range.location.row(), range.location.column()),
+    })
+}
+
+fn leading_space(line: &str) -> String {
+    line.chars()
+        .take_while(|char| char.is_whitespace())
+        .collect()
+}
+
+fn leading_words(line: &str) -> String {
+    line.trim()
+        .chars()
+        .take_while(|char| char.is_alphanumeric() || char.is_whitespace())
+        .collect()
+}
+
+fn suspected_as_section(line: &str) -> bool {
+    NUMPY_SECTION_NAMES_LOWERCASE.contains(&leading_words(line).to_lowercase().as_str())
+}
+
+#[derive(Debug)]
+pub struct SectionContext<'a> {
+    section_name: String,
+    previous_line: &'a str,
+    line: &'a str,
+    following_lines: &'a [&'a str],
+    original_index: usize,
+    is_last_section: bool,
+}
+
+/// Check if the suspected context is really a section header.
+fn is_docstring_section(context: &SectionContext) -> bool {
+    let section_name_suffix = context
+        .line
+        .trim()
+        .strip_prefix(&context.section_name)
+        .unwrap()
+        .trim();
+    let this_looks_like_a_section_name =
+        section_name_suffix == ":" || section_name_suffix.is_empty();
+    if !this_looks_like_a_section_name {
+        return false;
+    }
+
+    let prev_line = context.previous_line.trim();
+    let prev_line_ends_with_punctuation = [',', ';', '.', '-', '\\', '/', ']', '}', ')']
+        .into_iter()
+        .any(|char| prev_line.ends_with(char));
+    let prev_line_looks_like_end_of_paragraph =
+        prev_line_ends_with_punctuation || prev_line.is_empty();
+    if !prev_line_looks_like_end_of_paragraph {
+        return false;
+    }
+
+    true
+}
+
+/// Extract all `SectionContext` values from a docstring.
+pub fn section_contexts<'a>(lines: &'a [&'a str]) -> Vec<SectionContext<'a>> {
+    let suspected_section_indices: Vec<usize> = lines
+        .iter()
+        .enumerate()
+        .filter_map(|(lineno, line)| {
+            if lineno > 0 && suspected_as_section(line) {
+                Some(lineno)
+            } else {
+                None
+            }
+        })
+        .collect();
+
+    let mut contexts = vec![];
+    for lineno in suspected_section_indices {
+        let context = SectionContext {
+            section_name: leading_words(lines[lineno]),
+            previous_line: lines[lineno - 1],
+            line: lines[lineno],
+            following_lines: &lines[lineno + 1..],
+            original_index: lineno,
+            is_last_section: false,
+        };
+        if is_docstring_section(&context) {
+            contexts.push(context);
+        }
+    }
+
+    let mut truncated_contexts = vec![];
+    let mut end: Option<usize> = None;
+    for context in contexts.into_iter().rev() {
+        let next_end = context.original_index;
+        truncated_contexts.push(SectionContext {
+            section_name: context.section_name,
+            previous_line: context.previous_line,
+            line: context.line,
+            following_lines: if let Some(end) = end {
+                &lines[context.original_index + 1..end]
+            } else {
+                context.following_lines
+            },
+            original_index: context.original_index,
+            is_last_section: end.is_none(),
+        });
+        end = Some(next_end);
+    }
+    truncated_contexts.reverse();
+    truncated_contexts
+}
+
+fn check_blanks_and_section_underline(
+    checker: &mut Checker,
+    definition: &Definition,
+    context: &SectionContext,
+) {
+    let docstring = definition
+        .docstring
+        .expect("Sections are only available for docstrings.");
+
+    let mut blank_lines_after_header = 0;
+    for line in context.following_lines {
+        if !line.trim().is_empty() {
+            break;
+        }
+        blank_lines_after_header += 1;
+    }
+
+    // Nothing but blank lines after the section header.
+    if blank_lines_after_header == context.following_lines.len() {
+        if checker.settings.enabled.contains(&CheckCode::D407) {
+            checker.add_check(Check::new(
+                CheckKind::DashedUnderlineAfterSection(context.section_name.to_string()),
+                range_for(docstring),
+            ));
+        }
+        if checker.settings.enabled.contains(&CheckCode::D414) {
+            checker.add_check(Check::new(
+                CheckKind::NonEmptySection(context.section_name.to_string()),
+                range_for(docstring),
+            ));
+        }
+        return;
+    }
+
+    let non_empty_line = context.following_lines[blank_lines_after_header];
+    let dash_line_found = non_empty_line
+        .chars()
+        .all(|char| char.is_whitespace() || char == '-');
+
+    if !dash_line_found {
+        if checker.settings.enabled.contains(&CheckCode::D407) {
+            checker.add_check(Check::new(
+                CheckKind::DashedUnderlineAfterSection(context.section_name.to_string()),
+                range_for(docstring),
+            ));
+        }
+        if blank_lines_after_header > 0 {
+            if checker.settings.enabled.contains(&CheckCode::D212) {
+                checker.add_check(Check::new(
+                    CheckKind::NoBlankLinesBetweenHeaderAndContent(
+                        context.section_name.to_string(),
+                    ),
+                    range_for(docstring),
+                ));
+            }
+        }
+    } else {
+        if blank_lines_after_header > 0 {
+            if checker.settings.enabled.contains(&CheckCode::D408) {
+                checker.add_check(Check::new(
+                    CheckKind::SectionUnderlineAfterName(context.section_name.to_string()),
+                    range_for(docstring),
+                ));
+            }
+        }
+
+        if non_empty_line
+            .trim()
+            .chars()
+            .filter(|char| *char == '-')
+            .count()
+            != context.section_name.len()
+        {
+            if checker.settings.enabled.contains(&CheckCode::D409) {
+                checker.add_check(Check::new(
+                    CheckKind::SectionUnderlineMatchesSectionLength(
+                        context.section_name.to_string(),
+                    ),
+                    range_for(docstring),
+                ));
+            }
+        }
+
+        if checker.settings.enabled.contains(&CheckCode::D215) {
+            if leading_space(non_empty_line).len() > indentation(checker, docstring).len() {
+                checker.add_check(Check::new(
+                    CheckKind::SectionUnderlineNotOverIndented(context.section_name.to_string()),
+                    range_for(docstring),
+                ));
+            }
+        }
+
+        let line_after_dashes_index = blank_lines_after_header + 1;
+
+        if line_after_dashes_index < context.following_lines.len() {
+            let line_after_dashes = context.following_lines[line_after_dashes_index];
+            if line_after_dashes.trim().is_empty() {
+                let rest_of_lines = &context.following_lines[line_after_dashes_index..];
+                if rest_of_lines.iter().all(|line| line.trim().is_empty()) {
+                    if checker.settings.enabled.contains(&CheckCode::D414) {
+                        checker.add_check(Check::new(
+                            CheckKind::NonEmptySection(context.section_name.to_string()),
+                            range_for(docstring),
+                        ));
+                    }
+                } else {
+                    if checker.settings.enabled.contains(&CheckCode::D412) {
+                        checker.add_check(Check::new(
+                            CheckKind::NoBlankLinesBetweenHeaderAndContent(
+                                context.section_name.to_string(),
+                            ),
+                            range_for(docstring),
+                        ));
+                    }
+                }
+            }
+        } else {
+            if checker.settings.enabled.contains(&CheckCode::D414) {
+                checker.add_check(Check::new(
+                    CheckKind::NonEmptySection(context.section_name.to_string()),
+                    range_for(docstring),
+                ));
+            }
+        }
+    }
+}
+
+fn check_common_section(checker: &mut Checker, definition: &Definition, context: &SectionContext) {
+    let docstring = definition
+        .docstring
+        .expect("Sections are only available for docstrings.");
+
+    if checker.settings.enabled.contains(&CheckCode::D405) {
+        if !NUMPY_SECTION_NAMES.contains(&context.section_name.as_str())
+            && NUMPY_SECTION_NAMES.contains(titlecase(&context.section_name).as_str())
+        {
+            checker.add_check(Check::new(
+                CheckKind::CapitalizeSectionName(context.section_name.to_string()),
+                range_for(docstring),
+            ))
+        }
+    }
+
+    if checker.settings.enabled.contains(&CheckCode::D214) {
+        if leading_space(context.line).len() > indentation(checker, docstring).len() {
+            checker.add_check(Check::new(
+                CheckKind::SectionNotOverIndented(context.section_name.to_string()),
+                range_for(docstring),
+            ))
+        }
+    }
+
+    if context
+        .following_lines
+        .last()
+        .map(|line| !line.trim().is_empty())
+        .unwrap_or(true)
+    {
+        if context.is_last_section {
+            if checker.settings.enabled.contains(&CheckCode::D413) {
+                checker.add_check(Check::new(
+                    CheckKind::BlankLineAfterLastSection(context.section_name.to_string()),
+                    range_for(docstring),
+                ))
+            }
+        } else {
+            if checker.settings.enabled.contains(&CheckCode::D410) {
+                checker.add_check(Check::new(
+                    CheckKind::BlankLineAfterSection(context.section_name.to_string()),
+                    range_for(docstring),
+                ))
+            }
+        }
+    }
+
+    if checker.settings.enabled.contains(&CheckCode::D411) {
+        if !context.previous_line.is_empty() {
+            checker.add_check(Check::new(
+                CheckKind::BlankLineBeforeSection(context.section_name.to_string()),
+                range_for(docstring),
+            ))
+        }
+    }
+}
+
+fn check_missing_args(
+    checker: &mut Checker,
+    definition: &Definition,
+    docstrings_args: BTreeSet<&str>,
+) {
+    if let DefinitionKind::Function(parent)
+    | DefinitionKind::NestedFunction(parent)
+    | DefinitionKind::Method(parent) = definition.kind
+    {
+        if let StmtKind::FunctionDef {
+            args: arguments, ..
+        }
+        | StmtKind::AsyncFunctionDef {
+            args: arguments, ..
+        } = &parent.node
+        {
+            // Collect all the arguments into a single vector.
+            let mut all_arguments: Vec<&Arg> = arguments
+                .args
+                .iter()
+                .chain(arguments.posonlyargs.iter())
+                .chain(arguments.kwonlyargs.iter())
+                .skip(
+                    // If this is a non-static method, skip `cls` or `self`.
+                    if matches!(definition.kind, DefinitionKind::Method(_)) && !is_static(parent) {
+                        1
+                    } else {
+                        0
+                    },
+                )
+                .collect();
+            if let Some(arg) = &arguments.vararg {
+                all_arguments.push(arg);
+            }
+            if let Some(arg) = &arguments.kwarg {
+                all_arguments.push(arg);
+            }
+
+            // Look for arguments that weren't included in the docstring.
+            let mut missing_args: BTreeSet<&str> = Default::default();
+            for arg in all_arguments {
+                let arg_name = arg.node.arg.as_str();
+                if arg_name.starts_with('_') {
+                    continue;
+                }
+                if docstrings_args.contains(&arg_name) {
+                    continue;
+                }
+                missing_args.insert(arg_name);
+            }
+
+            if !missing_args.is_empty() {
+                let names = missing_args
+                    .into_iter()
+                    .map(String::from)
+                    .sorted()
+                    .collect();
+                checker.add_check(Check::new(
+                    CheckKind::DocumentAllArguments(names),
+                    Range::from_located(parent),
+                ));
+            }
+        }
+    }
+}
+
+fn check_parameters_section(
+    checker: &mut Checker,
+    definition: &Definition,
+    context: &SectionContext,
+) {
+    // Collect the list of arguments documented in the docstring.
+    let mut docstring_args: BTreeSet<&str> = Default::default();
+    let section_level_indent = leading_space(context.line);
+    for i in 1..context.following_lines.len() {
+        let current_line = context.following_lines[i - 1];
+        let current_leading_space = leading_space(current_line);
+        let next_line = context.following_lines[i];
+        if current_leading_space == section_level_indent
+            && (leading_space(next_line).len() > current_leading_space.len())
+            && !next_line.trim().is_empty()
+        {
+            let parameters = if let Some(semi_index) = current_line.find(':') {
+                // If the parameter has a type annotation, exclude it.
+                &current_line[..semi_index]
+            } else {
+                // Otherwise, it's just a list of parameters on the current line.
+                current_line.trim()
+            };
+            // Notably, NumPy lets you put multiple parameters of the same type on the same line.
+            for parameter in parameters.split(',') {
+                docstring_args.insert(parameter.trim());
+            }
+        }
+    }
+    // Validate that all arguments were documented.
+    check_missing_args(checker, definition, docstring_args);
+}
+
+pub fn check_numpy_section(
+    checker: &mut Checker,
+    definition: &Definition,
+    context: &SectionContext,
+) {
+    check_common_section(checker, definition, context);
+    check_blanks_and_section_underline(checker, definition, context);
+
+    if checker.settings.enabled.contains(&CheckCode::D406) {
+        let suffix = context
+            .line
+            .trim()
+            .strip_prefix(&context.section_name)
+            .unwrap();
+        if !suffix.is_empty() {
+            let docstring = definition
+                .docstring
+                .expect("Sections are only available for docstrings.");
+            checker.add_check(Check::new(
+                CheckKind::NewLineAfterSectionName(context.section_name.to_string()),
+                range_for(docstring),
+            ))
+        }
+    }
+
+    if checker.settings.enabled.contains(&CheckCode::D417) {
+        if titlecase(&context.section_name) == "Parameters" {
+            check_parameters_section(checker, definition, context);
+        }
+    }
+}

src/docstrings/types.rs

@@ -0,0 +1,23 @@
+use rustpython_ast::{Expr, Stmt};
+
+#[derive(Debug)]
+pub enum DefinitionKind<'a> {
+    Module,
+    Package,
+    Class(&'a Stmt),
+    NestedClass(&'a Stmt),
+    Function(&'a Stmt),
+    NestedFunction(&'a Stmt),
+    Method(&'a Stmt),
+}
+
+#[derive(Debug)]
+pub struct Definition<'a> {
+    pub kind: DefinitionKind<'a>,
+    pub docstring: Option<&'a Expr>,
+}
+
+pub enum Documentable {
+    Class,
+    Function,
+}

src/lib.rs

@@ -1,3 +1,4 @@
+#![allow(clippy::collapsible_if, clippy::collapsible_else_if)]
 use std::path::Path;
 
 use anyhow::Result;
@@ -17,7 +18,7 @@ mod check_lines;
 pub mod checks;
 pub mod cli;
 pub mod code_gen;
-pub mod docstrings;
+mod docstrings;
 pub mod fs;
 pub mod linter;
 pub mod logging;
@@ -28,6 +29,7 @@ pub mod printer;
 pub mod pyproject;
 mod python;
 pub mod settings;
+pub mod visibility;
 
 /// Run ruff over Python source code directly.
 pub fn check(path: &Path, contents: &str) -> Result<Vec<Message>> {

src/main.rs

@@ -81,9 +81,9 @@ fn read_from_stdin() -> Result<String> {
     Ok(buffer)
 }
 
-fn run_once_stdin(settings: &Settings, filename: &Path) -> Result<Vec<Message>> {
+fn run_once_stdin(settings: &Settings, filename: &Path, autofix: bool) -> Result<Vec<Message>> {
     let stdin = read_from_stdin()?;
-    let mut messages = lint_stdin(filename, &stdin, settings)?;
+    let mut messages = lint_stdin(filename, &stdin, settings, &autofix.into())?;
     messages.sort_unstable();
     Ok(messages)
 }
@@ -216,7 +216,9 @@ fn inner_main() -> Result<ExitCode> {
         Some(path) => debug!("Found project root at: {:?}", path),
         None => debug!("Unable to identify project root; assuming current directory..."),
     };
-    let pyproject = pyproject::find_pyproject_toml(&project_root);
+    let pyproject = cli
+        .config
+        .or_else(|| pyproject::find_pyproject_toml(&project_root));
     match &pyproject {
         Some(path) => debug!("Found pyproject.toml at: {:?}", path),
         None => debug!("Unable to find pyproject.toml; using default settings..."),
@@ -365,18 +367,20 @@ fn inner_main() -> Result<ExitCode> {
             println!("Formatted {modifications} files.");
         }
     } else {
-        let messages = if cli.files == vec![PathBuf::from("-")] {
-            if cli.fix {
-                eprintln!("Warning: --fix is not enabled when reading from stdin.");
-            }
-
+        let (messages, print_messages) = if cli.files == vec![PathBuf::from("-")] {
             let filename = cli.stdin_filename.unwrap_or_else(|| "-".to_string());
             let path = Path::new(&filename);
-            run_once_stdin(&settings, path)?
+            (
+                run_once_stdin(&settings, path, cli.fix)?,
+                !cli.quiet && !cli.fix,
+            )
         } else {
-            run_once(&cli.files, &settings, !cli.no_cache, cli.fix)?
+            (
+                run_once(&cli.files, &settings, !cli.no_cache, cli.fix)?,
+                !cli.quiet,
+            )
         };
-        if !cli.quiet {
+        if print_messages {
             printer.write_once(&messages)?;
         }
 

src/snapshots/ruff__linter__tests__C411_C411.py.snap

@@ -2,12 +2,12 @@
 source: src/linter.rs
 expression: checks
 ---
-- kind: NoNewLineAtEndOfFile
+- kind: UnnecessaryListCall
   location:
     row: 2
-    column: 9
+    column: 1
   end_location:
     row: 2
-    column: 9
+    column: 21
   fix: ~
 

src/snapshots/ruff__linter__tests__C413_C413.py.snap

@@ -0,0 +1,32 @@
+---
+source: src/linter.rs
+expression: checks
+---
+- kind:
+    UnnecessaryCallAroundSorted: list
+  location:
+    row: 2
+    column: 1
+  end_location:
+    row: 2
+    column: 16
+  fix: ~
+- kind:
+    UnnecessaryCallAroundSorted: reversed
+  location:
+    row: 3
+    column: 1
+  end_location:
+    row: 3
+    column: 20
+  fix: ~
+- kind:
+    UnnecessaryCallAroundSorted: reversed
+  location:
+    row: 4
+    column: 1
+  end_location:
+    row: 4
+    column: 34
+  fix: ~
+

src/snapshots/ruff__linter__tests__C414_C414.py.snap

@@ -0,0 +1,148 @@
+---
+source: src/linter.rs
+expression: checks
+---
+- kind:
+    UnnecessaryDoubleCastOrProcess:
+      - list
+      - list
+  location:
+    row: 2
+    column: 1
+  end_location:
+    row: 2
+    column: 14
+  fix: ~
+- kind:
+    UnnecessaryDoubleCastOrProcess:
+      - tuple
+      - list
+  location:
+    row: 3
+    column: 1
+  end_location:
+    row: 3
+    column: 15
+  fix: ~
+- kind:
+    UnnecessaryDoubleCastOrProcess:
+      - list
+      - tuple
+  location:
+    row: 4
+    column: 1
+  end_location:
+    row: 4
+    column: 15
+  fix: ~
+- kind:
+    UnnecessaryDoubleCastOrProcess:
+      - tuple
+      - tuple
+  location:
+    row: 5
+    column: 1
+  end_location:
+    row: 5
+    column: 16
+  fix: ~
+- kind:
+    UnnecessaryDoubleCastOrProcess:
+      - set
+      - set
+  location:
+    row: 6
+    column: 1
+  end_location:
+    row: 6
+    column: 12
+  fix: ~
+- kind:
+    UnnecessaryDoubleCastOrProcess:
+      - list
+      - set
+  location:
+    row: 7
+    column: 1
+  end_location:
+    row: 7
+    column: 13
+  fix: ~
+- kind:
+    UnnecessaryDoubleCastOrProcess:
+      - tuple
+      - set
+  location:
+    row: 8
+    column: 1
+  end_location:
+    row: 8
+    column: 14
+  fix: ~
+- kind:
+    UnnecessaryDoubleCastOrProcess:
+      - sorted
+      - set
+  location:
+    row: 9
+    column: 1
+  end_location:
+    row: 9
+    column: 15
+  fix: ~
+- kind:
+    UnnecessaryDoubleCastOrProcess:
+      - reversed
+      - set
+  location:
+    row: 10
+    column: 1
+  end_location:
+    row: 10
+    column: 17
+  fix: ~
+- kind:
+    UnnecessaryDoubleCastOrProcess:
+      - list
+      - sorted
+  location:
+    row: 11
+    column: 1
+  end_location:
+    row: 11
+    column: 16
+  fix: ~
+- kind:
+    UnnecessaryDoubleCastOrProcess:
+      - tuple
+      - sorted
+  location:
+    row: 12
+    column: 1
+  end_location:
+    row: 12
+    column: 17
+  fix: ~
+- kind:
+    UnnecessaryDoubleCastOrProcess:
+      - sorted
+      - sorted
+  location:
+    row: 13
+    column: 1
+  end_location:
+    row: 13
+    column: 18
+  fix: ~
+- kind:
+    UnnecessaryDoubleCastOrProcess:
+      - reversed
+      - sorted
+  location:
+    row: 14
+    column: 1
+  end_location:
+    row: 14
+    column: 20
+  fix: ~
+

src/snapshots/ruff__linter__tests__C416_C416.py.snap

@@ -0,0 +1,23 @@
+---
+source: src/linter.rs
+expression: checks
+---
+- kind:
+    UnnecessaryComprehension: list
+  location:
+    row: 2
+    column: 1
+  end_location:
+    row: 2
+    column: 15
+  fix: ~
+- kind:
+    UnnecessaryComprehension: set
+  location:
+    row: 3
+    column: 1
+  end_location:
+    row: 3
+    column: 15
+  fix: ~
+

src/snapshots/ruff__linter__tests__D100_D.py.snap

@@ -0,0 +1,13 @@
+---
+source: src/linter.rs
+expression: checks
+---
+- kind: PublicModule
+  location:
+    row: 1
+    column: 1
+  end_location:
+    row: 1
+    column: 1
+  fix: ~
+

src/snapshots/ruff__linter__tests__D101_D.py.snap

@@ -0,0 +1,13 @@
+---
+source: src/linter.rs
+expression: checks
+---
+- kind: PublicClass
+  location:
+    row: 14
+    column: 1
+  end_location:
+    row: 67
+    column: 1
+  fix: ~
+

src/snapshots/ruff__linter__tests__D102_D.py.snap

@@ -0,0 +1,29 @@
+---
+source: src/linter.rs
+expression: checks
+---
+- kind: PublicMethod
+  location:
+    row: 22
+    column: 5
+  end_location:
+    row: 25
+    column: 5
+  fix: ~
+- kind: PublicMethod
+  location:
+    row: 51
+    column: 5
+  end_location:
+    row: 54
+    column: 5
+  fix: ~
+- kind: PublicMethod
+  location:
+    row: 63
+    column: 5
+  end_location:
+    row: 67
+    column: 1
+  fix: ~
+

src/snapshots/ruff__linter__tests__D103_D.py.snap

@@ -0,0 +1,13 @@
+---
+source: src/linter.rs
+expression: checks
+---
+- kind: PublicFunction
+  location:
+    row: 395
+    column: 1
+  end_location:
+    row: 396
+    column: 1
+  fix: ~
+

src/snapshots/ruff__linter__tests__D105_D.py.snap

@@ -0,0 +1,13 @@
+---
+source: src/linter.rs
+expression: checks
+---
+- kind: MagicMethod
+  location:
+    row: 59
+    column: 5
+  end_location:
+    row: 62
+    column: 5
+  fix: ~
+

src/snapshots/ruff__linter__tests__D107_D.py.snap

@@ -0,0 +1,21 @@
+---
+source: src/linter.rs
+expression: checks
+---
+- kind: PublicInit
+  location:
+    row: 55
+    column: 5
+  end_location:
+    row: 58
+    column: 5
+  fix: ~
+- kind: PublicInit
+  location:
+    row: 529
+    column: 5
+  end_location:
+    row: 533
+    column: 1
+  fix: ~
+

src/snapshots/ruff__linter__tests__D201_D.py.snap

@@ -0,0 +1,23 @@
+---
+source: src/linter.rs
+expression: checks
+---
+- kind:
+    NoBlankLineBeforeFunction: 1
+  location:
+    row: 132
+    column: 5
+  end_location:
+    row: 132
+    column: 25
+  fix: ~
+- kind:
+    NoBlankLineBeforeFunction: 1
+  location:
+    row: 146
+    column: 5
+  end_location:
+    row: 146
+    column: 38
+  fix: ~
+

src/snapshots/ruff__linter__tests__D202_D.py.snap

@@ -0,0 +1,23 @@
+---
+source: src/linter.rs
+expression: checks
+---
+- kind:
+    NoBlankLineAfterFunction: 1
+  location:
+    row: 137
+    column: 5
+  end_location:
+    row: 137
+    column: 25
+  fix: ~
+- kind:
+    NoBlankLineAfterFunction: 1
+  location:
+    row: 146
+    column: 5
+  end_location:
+    row: 146
+    column: 38
+  fix: ~
+

src/snapshots/ruff__linter__tests__D203_D.py.snap

@@ -0,0 +1,32 @@
+---
+source: src/linter.rs
+expression: checks
+---
+- kind:
+    OneBlankLineBeforeClass: 0
+  location:
+    row: 156
+    column: 5
+  end_location:
+    row: 156
+    column: 33
+  fix: ~
+- kind:
+    OneBlankLineBeforeClass: 0
+  location:
+    row: 187
+    column: 5
+  end_location:
+    row: 187
+    column: 46
+  fix: ~
+- kind:
+    OneBlankLineBeforeClass: 0
+  location:
+    row: 521
+    column: 5
+  end_location:
+    row: 527
+    column: 8
+  fix: ~
+

src/snapshots/ruff__linter__tests__D204_D.py.snap

@@ -0,0 +1,23 @@
+---
+source: src/linter.rs
+expression: checks
+---
+- kind:
+    OneBlankLineAfterClass: 0
+  location:
+    row: 176
+    column: 5
+  end_location:
+    row: 176
+    column: 25
+  fix: ~
+- kind:
+    OneBlankLineAfterClass: 0
+  location:
+    row: 187
+    column: 5
+  end_location:
+    row: 187
+    column: 46
+  fix: ~
+

src/snapshots/ruff__linter__tests__D211_D.py.snap

@@ -0,0 +1,23 @@
+---
+source: src/linter.rs
+expression: checks
+---
+- kind:
+    NoBlankLineBeforeClass: 1
+  location:
+    row: 165
+    column: 5
+  end_location:
+    row: 165
+    column: 30
+  fix: ~
+- kind:
+    NoBlankLineBeforeClass: 1
+  location:
+    row: 176
+    column: 5
+  end_location:
+    row: 176
+    column: 25
+  fix: ~
+

src/snapshots/ruff__linter__tests__D214_sections.py.snap

@@ -2,12 +2,13 @@
 source: src/linter.rs
 expression: checks
 ---
-- kind: FitsOnOneLine
+- kind:
+    SectionNotOverIndented: Returns
   location:
-    row: 124
+    row: 135
     column: 5
   end_location:
-    row: 126
+    row: 141
     column: 8
   fix: ~
 

src/snapshots/ruff__linter__tests__D215_sections.py.snap

@@ -0,0 +1,23 @@
+---
+source: src/linter.rs
+expression: checks
+---
+- kind:
+    SectionUnderlineNotOverIndented: Returns
+  location:
+    row: 147
+    column: 5
+  end_location:
+    row: 153
+    column: 8
+  fix: ~
+- kind:
+    SectionUnderlineNotOverIndented: Returns
+  location:
+    row: 161
+    column: 5
+  end_location:
+    row: 165
+    column: 8
+  fix: ~
+

src/snapshots/ruff__linter__tests__D402_D.py.snap

@@ -0,0 +1,13 @@
+---
+source: src/linter.rs
+expression: checks
+---
+- kind: NoSignature
+  location:
+    row: 373
+    column: 5
+  end_location:
+    row: 373
+    column: 31
+  fix: ~
+

src/snapshots/ruff__linter__tests__D405_sections.py.snap

@@ -0,0 +1,23 @@
+---
+source: src/linter.rs
+expression: checks
+---
+- kind:
+    CapitalizeSectionName: returns
+  location:
+    row: 17
+    column: 5
+  end_location:
+    row: 23
+    column: 8
+  fix: ~
+- kind:
+    CapitalizeSectionName: Short summary
+  location:
+    row: 207
+    column: 5
+  end_location:
+    row: 221
+    column: 8
+  fix: ~
+

src/snapshots/ruff__linter__tests__D406_sections.py.snap

@@ -0,0 +1,41 @@
+---
+source: src/linter.rs
+expression: checks
+---
+- kind:
+    NewLineAfterSectionName: Returns
+  location:
+    row: 30
+    column: 5
+  end_location:
+    row: 36
+    column: 8
+  fix: ~
+- kind:
+    NewLineAfterSectionName: Raises
+  location:
+    row: 207
+    column: 5
+  end_location:
+    row: 221
+    column: 8
+  fix: ~
+- kind:
+    NewLineAfterSectionName: Returns
+  location:
+    row: 252
+    column: 5
+  end_location:
+    row: 262
+    column: 8
+  fix: ~
+- kind:
+    NewLineAfterSectionName: Raises
+  location:
+    row: 252
+    column: 5
+  end_location:
+    row: 262
+    column: 8
+  fix: ~
+

src/snapshots/ruff__linter__tests__D407_sections.py.snap

@@ -0,0 +1,50 @@
+---
+source: src/linter.rs
+expression: checks
+---
+- kind:
+    DashedUnderlineAfterSection: Returns
+  location:
+    row: 42
+    column: 5
+  end_location:
+    row: 47
+    column: 8
+  fix: ~
+- kind:
+    DashedUnderlineAfterSection: Returns
+  location:
+    row: 54
+    column: 5
+  end_location:
+    row: 58
+    column: 8
+  fix: ~
+- kind:
+    DashedUnderlineAfterSection: Raises
+  location:
+    row: 207
+    column: 5
+  end_location:
+    row: 221
+    column: 8
+  fix: ~
+- kind:
+    DashedUnderlineAfterSection: Returns
+  location:
+    row: 252
+    column: 5
+  end_location:
+    row: 262
+    column: 8
+  fix: ~
+- kind:
+    DashedUnderlineAfterSection: Raises
+  location:
+    row: 252
+    column: 5
+  end_location:
+    row: 262
+    column: 8
+  fix: ~
+

src/snapshots/ruff__linter__tests__D408_sections.py.snap

@@ -0,0 +1,14 @@
+---
+source: src/linter.rs
+expression: checks
+---
+- kind:
+    SectionUnderlineAfterName: Returns
+  location:
+    row: 85
+    column: 5
+  end_location:
+    row: 92
+    column: 8
+  fix: ~
+

src/snapshots/ruff__linter__tests__D409_sections.py.snap

@@ -0,0 +1,23 @@
+---
+source: src/linter.rs
+expression: checks
+---
+- kind:
+    SectionUnderlineMatchesSectionLength: Returns
+  location:
+    row: 99
+    column: 5
+  end_location:
+    row: 105
+    column: 8
+  fix: ~
+- kind:
+    SectionUnderlineMatchesSectionLength: Returns
+  location:
+    row: 207
+    column: 5
+  end_location:
+    row: 221
+    column: 8
+  fix: ~
+

src/snapshots/ruff__linter__tests__D410_sections.py.snap

@@ -0,0 +1,23 @@
+---
+source: src/linter.rs
+expression: checks
+---
+- kind:
+    BlankLineAfterSection: Returns
+  location:
+    row: 67
+    column: 5
+  end_location:
+    row: 78
+    column: 8
+  fix: ~
+- kind:
+    BlankLineAfterSection: Returns
+  location:
+    row: 207
+    column: 5
+  end_location:
+    row: 221
+    column: 8
+  fix: ~
+

src/snapshots/ruff__linter__tests__D411_sections.py.snap

@@ -0,0 +1,32 @@
+---
+source: src/linter.rs
+expression: checks
+---
+- kind:
+    BlankLineBeforeSection: Yields
+  location:
+    row: 67
+    column: 5
+  end_location:
+    row: 78
+    column: 8
+  fix: ~
+- kind:
+    BlankLineBeforeSection: Returns
+  location:
+    row: 122
+    column: 5
+  end_location:
+    row: 129
+    column: 8
+  fix: ~
+- kind:
+    BlankLineBeforeSection: Raises
+  location:
+    row: 207
+    column: 5
+  end_location:
+    row: 221
+    column: 8
+  fix: ~
+

src/snapshots/ruff__linter__tests__D412_sections.py.snap

@@ -0,0 +1,14 @@
+---
+source: src/linter.rs
+expression: checks
+---
+- kind:
+    NoBlankLinesBetweenHeaderAndContent: Short summary
+  location:
+    row: 207
+    column: 5
+  end_location:
+    row: 221
+    column: 8
+  fix: ~
+

src/snapshots/ruff__linter__tests__D414_sections.py.snap

@@ -0,0 +1,50 @@
+---
+source: src/linter.rs
+expression: checks
+---
+- kind:
+    NonEmptySection: Returns
+  location:
+    row: 54
+    column: 5
+  end_location:
+    row: 58
+    column: 8
+  fix: ~
+- kind:
+    NonEmptySection: Returns
+  location:
+    row: 67
+    column: 5
+  end_location:
+    row: 78
+    column: 8
+  fix: ~
+- kind:
+    NonEmptySection: Yields
+  location:
+    row: 67
+    column: 5
+  end_location:
+    row: 78
+    column: 8
+  fix: ~
+- kind:
+    NonEmptySection: Returns
+  location:
+    row: 161
+    column: 5
+  end_location:
+    row: 165
+    column: 8
+  fix: ~
+- kind:
+    NonEmptySection: Returns
+  location:
+    row: 252
+    column: 5
+  end_location:
+    row: 262
+    column: 8
+  fix: ~
+

src/snapshots/ruff__linter__tests__D417_sections.py.snap

@@ -0,0 +1,50 @@
+---
+source: src/linter.rs
+expression: checks
+---
+- kind:
+    DocumentAllArguments:
+      - y
+  location:
+    row: 389
+    column: 1
+  end_location:
+    row: 401
+    column: 1
+  fix: ~
+- kind:
+    DocumentAllArguments:
+      - test
+      - y
+      - z
+  location:
+    row: 425
+    column: 5
+  end_location:
+    row: 436
+    column: 5
+  fix: ~
+- kind:
+    DocumentAllArguments:
+      - test
+      - y
+      - z
+  location:
+    row: 440
+    column: 5
+  end_location:
+    row: 455
+    column: 5
+  fix: ~
+- kind:
+    DocumentAllArguments:
+      - a
+      - z
+  location:
+    row: 459
+    column: 5
+  end_location:
+    row: 471
+    column: 5
+  fix: ~
+

src/snapshots/ruff__linter__tests__D418_D.py.snap

@@ -0,0 +1,29 @@
+---
+source: src/linter.rs
+expression: checks
+---
+- kind: SkipDocstring
+  location:
+    row: 33
+    column: 5
+  end_location:
+    row: 37
+    column: 5
+  fix: ~
+- kind: SkipDocstring
+  location:
+    row: 85
+    column: 5
+  end_location:
+    row: 89
+    column: 5
+  fix: ~
+- kind: SkipDocstring
+  location:
+    row: 105
+    column: 1
+  end_location:
+    row: 110
+    column: 1
+  fix: ~
+

src/snapshots/ruff__linter__tests__F401_F401_2.py.snap

@@ -0,0 +1,6 @@
+---
+source: src/linter.rs
+expression: checks
+---
+[]
+

src/snapshots/ruff__linter__tests__F401_F401_3.py.snap

@@ -0,0 +1,6 @@
+---
+source: src/linter.rs
+expression: checks
+---
+[]
+