Skip to content

Move Odoo generic post-deploy adapter out of service#926

Merged
shiny-code-bot merged 1 commit into
mainfrom
refactor/odoo-generic-post-deploy-adapter
May 26, 2026
Merged

Move Odoo generic post-deploy adapter out of service#926
shiny-code-bot merged 1 commit into
mainfrom
refactor/odoo-generic-post-deploy-adapter

Conversation

@shiny-code-bot
Copy link
Copy Markdown
Collaborator

Summary

  • move the Odoo generic-web post-deploy adapter and driver-id selector into control_plane.workflows.odoo_generic_web_post_deploy
  • keep service.py as route orchestration that only asks for a post-deploy executor by driver id
  • move adapter behavior tests out of tests/test_service.py into a dedicated workflow test module

Validation

  • uv run python -m unittest tests.test_service
  • uv run python -m unittest tests.test_service tests.test_odoo_generic_web_post_deploy
  • uv run python -m unittest
  • uv run --extra dev ruff check --diff control_plane/service.py control_plane/workflows/odoo_generic_web_post_deploy.py tests/test_service.py tests/test_odoo_generic_web_post_deploy.py
  • uv run --extra dev ruff check control_plane/service.py control_plane/workflows/odoo_generic_web_post_deploy.py tests/test_service.py tests/test_odoo_generic_web_post_deploy.py
  • uv run --extra dev ruff format --check control_plane/service.py control_plane/workflows/odoo_generic_web_post_deploy.py tests/test_service.py tests/test_odoo_generic_web_post_deploy.py
  • uv run --extra dev mypy control_plane tests
  • uv run --extra dev ruff check .

Notes

  • uv run --extra dev ruff format --check . still reports existing unrelated repo-wide formatting drift in 45 files; touched files are formatted.
  • JetBrains changed-file inspection reported longstanding service.py findings unrelated to this refactor. The new workflow module is clean under ruff and mypy.

Refs #898
Refs #900

@shiny-code-bot shiny-code-bot merged commit 3ca7525 into main May 26, 2026
12 checks passed
@shiny-code-bot shiny-code-bot deleted the refactor/odoo-generic-post-deploy-adapter branch May 26, 2026 01:37
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.

2 participants