Contributors mailing list archives
Re: Project for special dynamic checks of maintainer-quality-toolsby
Tecnativa. S. L., Pedro M. Baeza
Yeah, I see that it's a usage a bit twisted. OK, let's wait what other contributors say.Regards.
2016-08-09 1:38 GMT+02:00 Moises Lopez <firstname.lastname@example.org>:
I thought that but the problem is that pylint-odoo is a pypi package for a python module.And we need a module odoo.Then, we don't need a 8.0, 9.0... versions of pylint-odooWe need a 8.0, 9.0... versions for runtime lints.Maybe, we can use the branch master for pylint-odoo and the other ones for runtime lints but I feel that we broken the fashion rules of github.What do you think?2016-08-08 18:08 GMT-05:00 Pedro M. Baeza (Tecnativa) <email@example.com>:Why not having them on pylint-odoo?Regards.2016-08-07 22:53 GMT+02:00 Moises Lopez <firstname.lastname@example.org>:We have modules that are useful forOCA/Maintainer-Quality-Tool
s(MQT) project.Demo xml_id referenced from data xml (odoolint module)If we have a "data/data.xml" where a ref="demo_xml_id" is used then will emit a warning.Our CI has a false green because we have installed with all demo.If you install the module with param "--without-demo=all" will fails with we have the error "External ID not found in the system"xml_id unreachable (odoolint module)We have cases where if you install a module without other ones will fail because we don't have a dependency added it.I have detected a case running odoolint module on OCA/l10n-spain:INFO openerp_test openerp.modules.loading: loading l10n_es_aeat_mod216/views/res_ partner_view.xmlWARNING openerp_test openerp.addons.odoolint.models .ir_model_data: The xml_id 'base_vat.view_partner_form' is unreachable.And using ~/odoo-8.0/odoo.py -i l10n_es_aeat_mod216 on new database empty.We have the error "External ID not found in the system"But our CI has a false green because we don't have installed one-by-one.Method too slow (odoo-profiler)We can get a profiler report of the methods of the project to get information of the top slow methods define in our project see imageThis is useful to detect performance issues early.If we have a method that is invoked many times or spent many time we can detect them in each PR with this module.SQL sentence too slow (runbot dockerized is required)Note: This change isn't posible added to travis because travis-ci#5784But with a environment isolated (like as runbot dockerized) is posible.We can get a pgbadger stats to detect slow queries or queries invoked many times, see imageThis is useful to detect performance issues early.* ConclusionWe could use OCA/server-tools but there are other ones what MQT don't use, then download all them just to use 4 ones is too slow to manage all builds of OCA.Well, we could use MQT project, but this project use a branch master and we require define a branch 7.0, 8.0, 9.0, master... ones to work fine with each version of odoo (like as a normal project of odoo modules)My proposal is to create a new project for manage this type of modules (I don't know maybe calling it "QA" modules).What do you think?--
Vauxoo, Moisés López Calderón