-
-
Notifications
You must be signed in to change notification settings - Fork 699
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
[ADD] 17.0 analysis #4294
[ADD] 17.0 analysis #4294
Conversation
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.
Minor remarks inline.
Thanks for initializing OpenUpgrade 17.0 !
This module is a containers of migration script to migrate from 15.0 to | ||
16.0 version. |
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.
This module is a containers of migration script to migrate from 15.0 to | |
16.0 version. | |
This module is a containers of migration script to migrate from 16.0 to | |
17.0 version. |
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.
Thanks, fixed now!
requirements.txt
Outdated
@@ -1 +1,2 @@ | |||
openupgradelib @ git+https://github.com/OCA/openupgradelib@master | |||
# generated from manifests external_dependencies | |||
openupgradelib |
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.
Why this change ?
As soon as merging a PR in openupgradelib doens't generate a new package on pypi, I guess that we should keep the url "openupgradelib @ git+https://github.com/OCA/openupgradelib@master"
Don't you think ?
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.
Thanks! Fixed by setting the external dependency in the module manifest to this git notation.
/ocabot migration openupgrade_scripts |
merged_models = { | ||
# odoo | ||
# OCA/... | ||
} |
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.
This file is empty. FYI, I began a list of renamed / merged Odoo modules here :
#4124 (comment)
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.
Thanks! Great work on the renamed and merge modules. I updated apriori with it and reran the analysis.
5561fa3
to
80705f3
Compare
…ylvain Le Gal Co-authored-by: Sylvain LE GAL <sylvain.legal@grap.coop>
80705f3
to
18f33f0
Compare
👍 |
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.
Thanks !
/ocabot merge nobump
On my way to merge this fine PR! |
@legalsylvain The merge process could not be finalized, because command
|
@sbidoul : maybe do you have an idea about the correct syntax to add git dependency for python package.
|
@legalsylvain it's
There is no way to declare the git URL in external dependencies. |
I mean, in V16 or before, there was a way to define alternative package in setup.py. There is a way to mention git url in pyproject.toml ? |
External dependencies overrides are still possible with pyproject.toml.
That (a git URL) was not possible with setup.py either. The only place we can do this is in |
Thanks a lot @sbidoul |
No description provided.