Contributors mailing list archives


Re: proposed mass commit on OCA repositories

Camptocamp France SAS, Alexandre Fayolle
- 20/07/2015 08:04:27
That's fine with me Raphaël. Thanks for the feedback.

Also please note that you don't have to have LINT_CHECK=1 in any of your environments, which means that travis_run_tests will not run flake8 on your code and you can ease the integration of l10n-brazil with the rest of the OCA code base (and save mass committers from special casing that repository...)




On 20/07/2015 00:23, Raphaël Valyi wrote:
<blockquote cite="" type="cite">
Hello Alexandre,

as for l10n-br, in 8.0 we can take your change. In 7.0, it's true that we have a non standard Travis setting. The reason is we used Travis before the OCA and later we didn't adapt to the OCA script as we found it more important to focus on v8 that is meeting the OCA requirements than fix PEP-8 in legacy 7.0. Thanks for the great work.

On Sun, Jul 19, 2015 at 10:37 AM, Alexandre Fayolle <> wrote:
Hello everyone,

Following the advice found on I'm planning to make a mass commit on the branches 6.1, 7.0, 8.0 of all OCA projects having .travis.yml with the following changes (except l10n-brazil):

+ sudo: false
+ cache: pip

I've tested this on server-tools and the speed up is noticeable (-25%)

If time allows, I will also try to extract the pip install <modulename> from the install section of .travis.yml and replace it with pip install -r requirements.txt which I think will make it easier to handle the external dependencies.

l10n-brazil cannot be processed for now because it uses a non standard travis set up. I'll open an issue on the project for the team to process).

If you think this is not a good idea, please speak up :-)


Post to:

Raphaël Valyi
Founder and consultant
+55 21 3942-2434

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