Skip to content

FIX: Indentation, ruff lint warnings and docs - #174

Merged
boscorat merged 3 commits into
masterfrom
cherry
Aug 6, 2026
Merged

FIX: Indentation, ruff lint warnings and docs#174
boscorat merged 3 commits into
masterfrom
cherry

Conversation

@boscorat

@boscorat boscorat commented Aug 6, 2026

Copy link
Copy Markdown
Owner

No description provided.

Jason Farrar added 3 commits August 6, 2026 12:24
- C408: dict() → {} in import_config.py
- C403: set([...]) → {...} in statements.py
- RUF015: [x][0] → next(...) in statement_functions.py
- RUF022: Sort __all__ in __init__.py and modules/__init__.py
- RUF023: Sort __slots__ in Statement, StatementBatch, TestHarness
- RUF034: Remove useless if-else ({} if x else {}) in statements.py
- RUF012: Add ClassVar annotations in housekeeping.py and test_cli.py
- SIM102: Combine nested if in statement_functions.py and test_docs.py
- SIM103: Inline return condition in statements.py
- PLW1510: Add check=False to subprocess.run in testing.py
- PYI034: Use Self return type for __enter__ in testing.py
- PIE810: Merge startswith calls in generate_docs.py
- PERF102: .items() → .values() in import_config.py and scripts
- DTZ005/DTZ001/DTZ007: Suppress intentional local-time uses with noqa
- BLE001/S110/S112: Suppress intentional broad exception catches with noqa
- Formatting fixes via ruff format

Signed-off-by: Jason Farrar <stillwaters.technology@gmail.com>
- Restore 'return db_secs' accidentally removed by ruff --fix in database.py
- Fix generate_docs.py exports guide to use double quotes (ruff format style)
- Add missing blank line in exports code example to match formatter output
- Regenerate all docs to match current generator output

Signed-off-by: Jason Farrar <stillwaters.technology@gmail.com>
The retain_descriptions parameter addition in d09e01c lost the 4-space
indentation of the function body, placing the if/return statements at
module level and causing a SyntaxError on import.

Signed-off-by: Jason Farrar <stillwaters.technology@gmail.com>
@boscorat
boscorat merged commit 02f6bec into master Aug 6, 2026
5 checks passed
@boscorat
boscorat deleted the cherry branch August 6, 2026 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant