Contributors mailing list archives

contributors@odoo-community.org

Browse archives

Avatar

Re: Codacy usage

by
Camptocamp France SAS, Alexandre Fayolle
- 22/05/2017 16:06:26
On closer inspection, I think Daniel Reis could be the one who added it.

Daniel, is it ok with you if I remove the web hook?

Alexandre

On 22/05/2017 15:54, Pedro Manuel Baeza Romero wrote:
> I have just said the same on my PR:
> https://github.com/OCA/project/pull/187#issuecomment-303087341
> 
> +1 for deactivating it.
> 
> Regards.
> 
> 2017-05-22 15:23 GMT+02:00 Alexandre Fayolle
> <alexandre.fayolle@camptocamp.com
> <mailto:alexandre.fayolle@camptocamp.com>>:
> 
>     Hello
> 
>     I see some projects such as OCA/project are using Codacy as part of the
>     webhooks. The net effect is that all PRs are red because codacy does not
>     know about __manifest__.py, unused imports and such. This makes
>     reviewing the PRs harder than needed IMO
> 
>     I'm therefore questionning the decision behind the use of this tool, at
>     least without a configuration which filters out the blatant false
>     positives.
> 
> 
> 
> 
>     -- 
>     Alexandre Fayolle
>     Chef de Projet
>     Tel : +33 4 58 48 20 30 
> 
>     Camptocamp France SAS
>     Savoie Technolac, BP 352
>     73377 Le Bourget du Lac Cedex
>     http://www.camptocamp.com
> 
>     _______________________________________________
>     Mailing-List: https://odoo-community.org/groups/contributors-15
>     
>     Post to: mailto:contributors@odoo-community.org
>     <mailto:contributors@odoo-community.org>
>     Unsubscribe: https://odoo-community.org/groups?unsubscribe
>     
> 
> 
> _______________________________________________
> Mailing-List: https://odoo-community.org/groups/contributors-15
> Post to: mailto:contributors@odoo-community.org
> Unsubscribe: https://odoo-community.org/groups?unsubscribe
> 


-- 
Alexandre Fayolle
Chef de Projet
Tel : +33 4 58 48 20 30

Camptocamp France SAS
Savoie Technolac, BP 352
73377 Le Bourget du Lac Cedex
http://www.camptocamp.com

Reference