Skip to content

[13.0][FIX] Ask for unversioned openupgradelib in requirements - #3029

Merged
pedrobaeza merged 1 commit into
OCA:13.0from
Tecnativa:13.0-fix-dependecies-oulib
Dec 27, 2021
Merged

[13.0][FIX] Ask for unversioned openupgradelib in requirements#3029
pedrobaeza merged 1 commit into
OCA:13.0from
Tecnativa:13.0-fix-dependecies-oulib

Conversation

@joao-p-marques

@joao-p-marques joao-p-marques commented Dec 27, 2021

Copy link
Copy Markdown
Member

Asking for a specific openupgradelib version from master is very limiting, as installing OpenUpgrade will likely override any different version you might want to have in your environment.

Instead, we should ask for the dependency from a generic version (or, if a minimal version is necessary, use semver standards instead) and allow the user ti install another specific version if needed.

@Tecnativa

ping @pedrobaeza

Asking for a specific `openupgradelib` version from `master` is very limiting, as installing OpenUpgrade will likely override any different version you might want to have in your environment.

Instead, we should ask for the dependency from a generic version (or, if a minimal version is necessary, use semver standards instead) and allow the user ti install another specific version if needed.

@Tecnativa
@pedrobaeza

Copy link
Copy Markdown
Member

Now that we are versioning very quick, it makes totally sense, so merging.

@pedrobaeza
pedrobaeza merged commit 25121b8 into OCA:13.0 Dec 27, 2021
@pedrobaeza
pedrobaeza deleted the 13.0-fix-dependecies-oulib branch December 27, 2021 17:22

@legalsylvain legalsylvain left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thansk.

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