Contributors mailing list archives

contributors@odoo-community.org

Avatar

Re: proposed mass commit on OCA repositories

by
rvalyi@akretion.com
- 19/07/2015 22:13:00
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 <alexandre.fayolle@camptocamp.com> wrote:
Hello everyone,

Following the advice found on http://docs.travis-ci.com/user/migrating-from-legacy/ 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 :-)

--
Alexandre


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




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


Reference