Contributors mailing list archives

contributors@odoo-community.org

Browse archives

Avatar

Re: OCA Coding Standards and recommended IDEs

by
Open Source Integrators, Maxime Chambreuil
- 19/10/2015 00:14:22
Sorry Oleg, most of his work is about the bzr support in PyCharm...

Cheers,
--
Maxime Chambreuil
+1 (514) 276-5468 #126

----- Le 16 Oct 15, à 9:23, oleg kuryan <oleg.kuryan@xpansa.com> a écrit :

@Daniel, agree that command line tool is what is needed. 
It will be cool, if we can do very simple like this when new:
- Here is link to OCA coding standards ...
- Here is plugins for popular editors that check onsave of the file all possible coding standards
- here is link to snippets library that can be embedded in popular editors
Now all possible is done for kick start and developer is ready to contribute. 

@Maxime, I will be happy to see Sandy's work. Can you share it?



///

Best Regards,
Oleg Kuryan

Technical Director, XPANSA Group | ERP, BI, E-commerce, Data Mining and DMS consulting
Xpansa has developed first full Odoo Android App called mERP. Download at https://play.google.com/merp
/// site  : xpansa.com
/// mail  : oleg.kuryan@xpansa.com
/// phone : +375293358638
/// skype : kuryan.oleg

On Fri, Oct 16, 2015 at 3:53 PM, Maxime Chambreuil <maxime.chambreuil@savoirfairelinux.com> wrote:
Without pushing or recommending for a particular IDE, the OCA could have a repository under the Tools Maintainers PSC for each one allowing anyone willing to improve its tools to collaborate with others.

Having an Odoo plugin (like pylint-odoo or Magento module for example) for IDE X in the X platform of sharing plugins could encourage X users to hear about Odoo and join us. We should favor interactions with other communities.

Oleg, Sandy (bwrsandman) made some stuff that we can share.

Following the new organization, this is a decision left to the Tools Maintainers PSC (hence the cc to tools).

My 2 cents,
--
Maxime Chambreuil
+1 (514) 276-5468 #126

----- Le 16 Oct 15, à 7:23, oleg kuryan <oleg.kuryan@xpansa.com> a écrit :

Eric,
Agree. The main idea behind of suggestion - is improving quality and quantity of iterations need to review.
Idea with limitations on supported editors just allows to build and maintain base of snippets for them. For example, for Sublime they already exists, so no big job to review them and correct, if needed to OCA needs. 
Maybe already there are for other Editors (somebody created for personal use and willing to share). 

And yes, that is something that will be good to have under maintainer tools.

///

Best Regards,
Oleg Kuryan

Technical Director, XPANSA Group | ERP, BI, E-commerce, Data Mining and DMS consulting
Xpansa has developed first full Odoo Android App called mERP. Download at https://play.google.com/merp
/// site  : xpansa.com
/// mail  : oleg.kuryan@xpansa.com
/// phone : +375293358638
/// skype : kuryan.oleg

On Fri, Oct 16, 2015 at 3:08 PM, Eric Caudal <eric.caudal@elico-corp.com> wrote:

It could be interesting to provide/share some snipet files in maintainer-tools for example: it would increase the quality of received PR.
I am not in favor of pushing one or another tool

--
Eric Caudal [Founder and CEO]
Skype: elico.corp. Phone: + 86 186 2136 1670 (Cell), + 86 21 6211 8017/27/37 (Office)
Elico Shanghai (Shenzhen/Singapore) Odoo Gold Partner, best Odoo Partner 2014 for APAC
On 10/16/2015 07:38 PM, Yajo wrote:
<blockquote cite="mid:CAHceOEffrMQZ_Tx8BY9fTx-hKoR3-KD0HMzGCmLHGm3ZYUJakQ@mail.gmail.com" type="cite">

2015-10-16 13:23 GMT+02:00 Leonardo Pistone <lpistone@gmail.com>:
I disagree: I don't think we should support / encourage specific IDEs or other tools. These are very personal choices.

Same here. Please don't do it.

As for tools, I agree that we'd need a simple command line wrapper that lints according to the OCA conventions. Then anyone will be able to tell their editor of choice to use it instead of i.e. "flake8".

Everyone is free to use a text editor that has no flake8 support, so adding those checks you say in travis would be wiser IMHO, as it is now with flake8.


_______________________________________________
Mailing-List: http://odoo-community.org/groups/contributors-15
Post to: mailto:contributors@odoo-community.org
Unsubscribe: http://odoo-community.org/groups?unsubscribe


_______________________________________________
Mailing-List: http://odoo-community.org/groups/contributors-15
Post to: mailto:contributors@odoo-community.org
Unsubscribe: http://odoo-community.org/groups?unsubscribe


_______________________________________________
Mailing-List: http://odoo-community.org/groups/contributors-15
Post to: mailto:contributors@odoo-community.org
Unsubscribe: http://odoo-community.org/groups?unsubscribe


_______________________________________________
Mailing-List: http://odoo-community.org/groups/contributors-15
Post to: mailto:contributors@odoo-community.org
Unsubscribe: http://odoo-community.org/groups?unsubscribe


Reference