OCA / l10n-france

France Localization for Odoo
GNU Affero General Public License v3.0
42 stars 117 forks source link

Migration to version 18.0 #573

Open OCA-git-bot opened 5 days ago

OCA-git-bot commented 5 days ago

Todo

https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-18.0

Modules to migrate

Missing module? Check https://github.com/OCA/maintainer-tools/wiki/%5BFAQ%5D-Missing-modules-in-migration-issue-list

alexis-via commented 1 day ago

Following a discusson at Odoo Experience, we plan to adopt another method on the 18.0 branch: push the 17.0 branch with modules marked as installable=False (while keeping the config files of the 18.0 branch). It has many advantages: lighter branches, easier migration PR and several OCA projects have gone back to this good old method (for example : OCA/mis-builder). If you disagree with this, give your reasons here before tonight!

pedrobaeza commented 1 day ago

I don't see it as an advantage, as there may be missing commits from the previous branch (17.0 in this case) in the moment of migrating a module (because between the moment creating the 18.0 branch from 17.0) till the moment of migrating the module, other commits may enter in the previous branch, and thus, not being aware of them. The other method is always the same for getting the latest commit history.

It also pollutes the commit history with commits doing/undoing things, and provoke conflicts, as non installable modules create an entry on the pre-commit config file (example: https://github.com/OCA/mis-builder/blob/a4796288344b4ada3ea77e1d810221696f98449a/.pre-commit-config.yaml#L4), and each time you merge a migration PR, it may require a rebase on all the other PRs fixing the conflict.

Only mis-builder and queue repos are using them. I totally dislike such technique, but it's more reasonable on that repos, which have very few modules. Applying it to repos like this with a lot of modules will cause several troubles.

Anyway, I don't have vote here, but just for letting you know the disadvantages for taking an informed decision.