Skip to content

feat: add --project-dir option to create command#11

Closed
nilshamerlinck wants to merge 1 commit intotrobz:mainfrom
nilshamerlinck:project_dir
Closed

feat: add --project-dir option to create command#11
nilshamerlinck wants to merge 1 commit intotrobz:mainfrom
nilshamerlinck:project_dir

Conversation

@nilshamerlinck
Copy link
Contributor

Summary

  • Adds --project-dir option that uses odoo-addons-path to auto-detect --addons-path, --odoo-dir, and --preset=project
  • Makes ODOO_VERSION optional when --project-dir is used — inferred from odoo/release.py
  • Adds odoo-addons-path as a dependency

Usage

# Before: all parameters manual
odoo-venv create 18.0 --preset project --odoo-dir /path/to/project/odoo --addons-path "..."

# After: single flag does it all
odoo-venv create --project-dir /path/to/project

Test plan

  • odoo-venv create --project-dir /path/to/project --dry-run — auto-detects everything
  • odoo-venv create 18.0 --project-dir /path --dry-run — explicit version takes precedence
  • odoo-venv create --preset local --project-dir /path --dry-run — explicit preset takes precedence
  • odoo-venv create --dry-run — errors with clear message about missing ODOO_VERSION
  • odoo-venv create 18.0 --dry-run — existing behavior unchanged

🤖 Generated with Claude Code

…sion

Uses odoo-addons-path to detect codebase layout and infer --addons-path,
--odoo-dir, --python-version, and --preset=project from a project directory.
ODOO_VERSION becomes optional when --project-dir is provided, as it is
read from odoo/release.py.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@nilshamerlinck
Copy link
Contributor Author

  • waiting for F#T67239
  • and for --create-launcher, which will behave a little bit differently when we have --project-dir: launcher will execute odoo-addons-path to obtain a dynamic addons_path at each launch

@nilshamerlinck
Copy link
Contributor Author

superseded by #12

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