-
-
Notifications
You must be signed in to change notification settings - Fork 785
[MIG] Bootstrapping 16.0 + analysis #3583
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
Changes from 3 commits
02ec4f7
e10af58
f48ec23
a060a76
87a301b
9b58ca8
1999340
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,25 @@ | ||
| """ Encode any known changes to the database here | ||
| to help the matching process | ||
| """ | ||
|
|
||
| # Renamed modules is a mapping from old module name to new module name | ||
| renamed_modules = { | ||
| # odoo | ||
| # OCA/... | ||
| } | ||
|
|
||
| # Merged modules contain a mapping from old module names to other, | ||
| # preexisting module names | ||
| merged_modules = { | ||
| # odoo | ||
| # OCA/... | ||
| } | ||
|
|
||
| # only used here for upgrade_analysis | ||
| renamed_models = { | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The analysis has to be re-run if renamed models are found. Do you know if there's any right now?
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. No, not aware of any.
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The re-run will have to be done few months later, anyway. |
||
| # odoo | ||
| # OCA/... | ||
| } | ||
|
|
||
| # only used here for upgrade_analysis | ||
| merged_models = {} | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1 @@ | ||
| This module is a containers of migration script to migrate from 14.0 to 15.0 version. | ||
| 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.
Why changing uppercase?
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.
Oh I did not pay any attention to the casing when I ran oca-gen-addon-readme. I didn't know it mattered, and I'm still slightly annoyed the project name has to be passed on the command line 😛
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.
It's for reducing the diff essentially, and yes, you need to introduce it, as it can't be deduced from the folder itself.
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.
Couldn't we introduce a .oca-project-name file in the root of the addons directory?
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.
That's a good idea, but it should be proposed at maintainer-tools, and maybe harmonize it with ocaport hidden directory (not sure if it's
.oca), and also bootstrap it through the copier template.