-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[14.0][FIX] *: remove lxml
from python depends
#2612
Conversation
0a71a1c
to
249891a
Compare
Such dependency is already included in Odoo requirements using a pinned version [1]. Adding here could cause to upgrade the library to an incompatible version. [1] https://github.com/odoo/odoo/blob/54e58b3e47ee/requirements.txt#L23
Such dependency is already included in Odoo requirements using a pinned version [1]. Adding here could cause to upgrade the library to an incompatible version. [1] https://github.com/odoo/odoo/blob/54e58b3e47ee/requirements.txt#L23
lxml
from dependslxml
from python depends
249891a
to
9704d72
Compare
Hi @moylop260, Could you review, please? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Hi @pedrobaeza, what do you think about this one? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/ocabot merge patch
lxml
from python dependslxml
from python depends
On my way to merge this fine PR! |
This PR has the |
Congratulations, your PR was merged at e12761b. Thanks a lot for contributing to OCA. ❤️ |
Such dependency is already included in Odoo requirements using a pinned
version [1]. Adding here could cause to upgrade the library to an
incompatible version.
[1] https://github.com/odoo/odoo/blob/54e58b3e47ee/requirements.txt#L23