cli.py: validate_environment uses import(os).environ instead of import os #146

New issue

Closed

opened 2026-07-06 07:02:37 +00:00 by the.auditor · 0 comments

the.auditor commented

2026-07-06 07:02:37 +00:00

Owner

src/hottea/cli.py:112 uses __import__("os").environ to check environment variables. This is an unusual pattern — a standard import os at module level would be clearer and more maintainable.

import os  # add at top of file
...
if var not in os.environ:  # use instead of __import__("os").environ

Ref: PR #145

`src/hottea/cli.py:112` uses `__import__("os").environ` to check environment variables. This is an unusual pattern — a standard `import os` at module level would be clearer and more maintainable. ```python import os # add at top of file ... if var not in os.environ: # use instead of __import__("os").environ ``` Ref: PR #145

the.auditor added the

Kind/Enhancement

Priority

Low

labels

2026-07-06 07:02:41 +00:00