-
Notifications
You must be signed in to change notification settings - Fork 61
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
click-odoo-update: new CLI option to control files that are part of the modules hash #83
Comments
Yes it is necessary because python files contain model and field definitions that need a database upgrade when changed. |
Oh right, I totally missed that. But it would be nice if there was an option --ignore-py in case of minor changes that dont contain models or fields definition, don't you think ? |
There is already a system parameter in the database to control which files are part of the hash: click-odoo-contrib/click_odoo_contrib/update.py Lines 26 to 27 in d7f8c32
It could be interesting to control it with a CLI option too -- although it is a potential footgun :). |
Merci stéphane 👍 |
If it's okay I'll work on a PR for this feature. |
@yassinelachgar that is very welcome, thank you. I might be interesting to discuss the CLI option first (i.e. update the documentation in the README before coding). Also, when it comes to it, don't forget tests. |
As far as I know, changes in .py files does not imply a module update for the changes to work. Would it not be optimal to detect changes only in XML files ?
The problem here is that a simple change in .py file triggers the module updates which takes additional time when running the server.
The text was updated successfully, but these errors were encountered: