Skip to content
This repository was archived by the owner on May 3, 2026. It is now read-only.

Fix import ordering in service module#208

Merged
brothercorvo merged 1 commit intomainfrom
corvo/fix-module-level-import-order
Jan 8, 2026
Merged

Fix import ordering in service module#208
brothercorvo merged 1 commit intomainfrom
corvo/fix-module-level-import-order

Conversation

@brothercorvo
Copy link
Copy Markdown
Contributor

Motivation

  • Address lint warnings about module-level imports not being at the top of the file in reticulum_openapi/service.py.
  • Ensure the side-effect-only import for multiprocessing compatibility is retained while keeping the import block contiguous.

Description

  • Moved the from . import _multiprocessing_compat as _mp_compat import into the main import block so module-level imports are contiguous.
  • Relocated the side-effect retention assignment _ = _mp_compat to the end of the import section to preserve the import’s side effects for linters.
  • No functional logic was changed; only import ordering and the position of the side-effect assignment were updated in service.py.

Testing

  • No automated tests were executed as part of this change.
  • Static linting (flake8) was the motivating check for the edit, but it was not re-run here.
  • Unit tests were not run.
  • Integration tests were not run.

Codex Task

@brothercorvo brothercorvo merged commit 8b5563f into main Jan 8, 2026
0 of 2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant