Skip to content

Conversation

@sbidoul
Copy link
Member

@sbidoul sbidoul commented Nov 9, 2025

No description provided.

@sbidoul sbidoul requested review from Copilot and petrus-v November 9, 2025 22:45
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR upgrades the click-odoo-contrib dependency from version 1.20 to 1.23 and leverages new features to simplify database initialization and configuration management.

  • Updated click-odoo-contrib to version 1.23 with a minimum version constraint of >=1.22
  • Simplified the odoo-update entrypoint script using the new --unless-initialized flag
  • Removed explicit config file references, relying on environment variable configuration

Reviewed Changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

File Description
uv.lock Updates click-odoo-contrib package from version 1.20 to 1.23 with new hashes and metadata
pyproject.toml Adds version constraint >=1.22 for click-odoo-contrib dependency
entrypoints/000_15_odoo-update Simplifies database initialization logic by using click-odoo-initdb --unless-initialized flag instead of manual checks
docker-compose.yml Removes explicit -c /etc/odoo.cfg flag from odoo command, relying on default configuration handling

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@sbidoul sbidoul force-pushed the 14.0-simplify-init-update-sbi branch from 5671f19 to b526d4e Compare November 9, 2025 22:50
- use new --unless-initialized click-odoo-initdb option
- don't invoke click-odoo-update twice
@sbidoul sbidoul force-pushed the 14.0-simplify-init-update-sbi branch from b526d4e to 8d330e2 Compare November 9, 2025 22:51
Copy link

@petrus-v petrus-v left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for reducing this technical debt !

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