Skip to content

[FIX] Modify pipx inject command in README#93

Closed
rrebollo wants to merge 1 commit into
OCA:mainfrom
rrebollo:patch-1
Closed

[FIX] Modify pipx inject command in README#93
rrebollo wants to merge 1 commit into
OCA:mainfrom
rrebollo:patch-1

Conversation

@rrebollo

Copy link
Copy Markdown

Updated pipx inject command. Current instructions lead me to:

$ pipx inject --include-deps  oca-port git+https://github.com/OCA/odoo-module-migrator.git@master
Cannot pass --include-deps if --include-apps is not passed as well

Updated pipx command to include apps for oca-port.

@edescalona edescalona left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@sebalix

sebalix commented Jan 9, 2026

Copy link
Copy Markdown
Collaborator

Hello, thanks for this PR, but I cannot reproduce the issue locally, I tested with pipx version 1.4.3, and the latest 1.8.0.

My guess is you are using an older version of pipx, as the error message has been removed by this PR to make --include-apps implicit:

Can you try by upgrading your pipx installation?

@rrebollo

Copy link
Copy Markdown
Author

I think you are right. I don't remember it but it looks like I did upgrade pipx recently because I just test the installation instructions again and it's working as expected. So I'm closing this. Thank you.

@rrebollo rrebollo closed this Jan 10, 2026
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.

3 participants