GeezerVison is showing github.com — Use one-line module docstrings, which Codacy's D212 and D213 both accept · EngineScript/EngineScript@c710eee
File tree
Expand file treeCollapse file tree
Open diff view settings
Expand file treeCollapse file tree
Open diff view settings
Collapse file
Copy file name to clipboardExpand all lines: .github/scripts/check-deployment-surface.py +6-8Lines changed: 6 additions & 8 deletions | Original file line number | Diff line number | Diff line change |
|---|
|
1 | 1 | #!/usr/bin/env python3 |
2 | | -""" |
3 | | -Verify the files under the panel's deployed directories against a tracked allowlist. |
4 | | -
|
5 | | -The installer copies config/var/www/admin/control-panel/ into the web root with |
6 | | -`cp -a`, and copies two named files from config/var/www/admin/tools/ into the |
7 | | -tools it installs. Every file in those directories is served, so nothing else |
8 | | -may live there: no test, fixture, development file, archive, or executable. |
9 | | -""" |
| 2 | +"""Verify the files under the panel's deployed directories against a tracked allowlist.""" |
| 3 | + |
| 4 | +# The installer copies config/var/www/admin/control-panel/ into the web root with |
| 5 | +# `cp -a`, and copies two named files from config/var/www/admin/tools/ into the |
| 6 | +# tools it installs. Every file in those directories is served, so nothing else |
| 7 | +# may live there: no test, fixture, development file, archive, or executable. |
10 | 8 |
|
11 | 9 | from __future__ import annotations |
12 | 10 |
|
|
Collapse file
Copy file name to clipboardExpand all lines: .github/scripts/test-automation.py +2-4Lines changed: 2 additions & 4 deletions | Original file line number | Diff line number | Diff line change |
|---|
|
1 | 1 | #!/usr/bin/env python3 |
2 | | -""" |
3 | | -Standard-library regression fixtures for the repository's automation scripts. |
| 2 | +"""Standard-library regression fixtures for the repository's automation scripts.""" |
4 | 3 |
|
5 | | -All mutation is confined to a temporary fixture repository on the runner. |
6 | | -""" |
| 4 | +# All mutation is confined to a temporary fixture repository on the runner. |
7 | 5 |
|
8 | 6 | from __future__ import annotations |
9 | 7 |
|
|
Collapse file
Copy file name to clipboardExpand all lines: .ruff.toml +2-3Lines changed: 2 additions & 3 deletions | Original file line number | Diff line number | Diff line change |
|---|
@@ -7,10 +7,9 @@ target-version = "py312" |
7 | 7 | [lint] |
8 | 8 | select = ["ALL"] |
9 | 9 | ignore = [ |
10 | | - # Two pairs of docstring rules contradict each other; D211 and D213 are the ones kept. |
11 | | - # D213 (the summary starts on the line after the opening quotes) is the one Codacy checks. |
| 10 | + # Two pairs of docstring rules contradict each other; D211 and D212 are the ones kept. |
12 | 11 | "D203", |
13 | | - "D212", |
| 12 | + "D213", |
14 | 13 | # These two conflict with the formatter. |
15 | 14 | "COM812", |
16 | 15 | "ISC001", |
|