Community mailing list archives
Re: Proposal to review the OCA contribution guidelinesby
Dear Odoo community members,
I have been following a number of mail threads recently and I think that we are navigating in the wrong direction.
I’ll make myself clear by rephrasing the OCA mission statement and an example of where it goes wrong.
The OCA mission (cf. http://odoo-community.org/ ):
“The Odoo Community Association, or OCA, is a nonprofit organization whose mission is to support the collaborative development of Odoo features and promote its widespread use.”
My understanding of ‘features’ is ‘functionality’.
Our focus should be on stimulating the community to contribute new functionality and to improve existing functionality.
Based upon a few mail threads, we are currently adding barriers to contribution in stead of stimulating it.
I’ll illustrate my point via the following thread: OCA/maintainer-quality-tools#46.
The OCA guideline is that new contributions should be PEP8 compliant.
This guideline adds an extra barrier to people who want to contribute modules.
Why should we reject modules that comply to the following criteria
- The module adds extra functionality
- The module is compliant with the coding practices of the EDITOR (not the OCA)
- The module is compatible with the standard modules from Odoo
We are adding modules on top of the Odoo suite of business applications. None of the modules from Odoo are PEP8 compliant.
The vast majority of the current OCA modules are not PEP8 compliant.
Since I am working with OpenERP/Odoo (5 years now) I have never seen an RFP nor a customer demand to deliver a PEP8 compliant solution.
Hence why should we add this extra barrier ?
I personally find it hard to motivate myself to restyle a module that is working fine for many years in order to make it PEP8 compliant (with the ‘punch card age’ line length limitation as the most extreme one since I don’t like to spend even a minute of my time in order to make code harder to read and maintain).
Luc De Meyer
Rusatiralaan 1, 1083 Brussel
<img border=0 width=96 height=33 id="Picture_x0020_1" src="cid:image001.jpg@01CFB6FE.C4114BD0" alt="cid:image002.jpg@01CE271A.FF53FB90">
Luc De Meyer