Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[IMP] Travis con flake8/pylint separado #94

Merged
merged 1 commit into from
Dec 18, 2014

Conversation

pedrobaeza
Copy link
Member

No description provided.

@yvaucher
Copy link
Member

Missing some dependencies ?

'Error', u"You try to install module 'l10n_es_aeat_mod340' that depends on module 'account_chart_update'.\nBut the latter module is not available in your system."

@pedrobaeza
Copy link
Member Author

Yeah, that's it. I remove a private repo that contains temporarily the module, but now I see that OCA/account-financial-tools#92 is not still merged. As I see two approvals, I'm going to merge this one and restart Travis.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 2589637 on pedrobaeza:8.0-travis_lint into 31a90dd on OCA:8.0.

@pedrobaeza
Copy link
Member Author

@yvaucher Green!

@guewen
Copy link
Member

guewen commented Dec 18, 2014

👍

1 similar comment
@yvaucher
Copy link
Member

👍

yvaucher added a commit that referenced this pull request Dec 18, 2014
[IMP] Travis con flake8/pylint separado
@yvaucher yvaucher merged commit 46b0914 into OCA:8.0 Dec 18, 2014
@pedrobaeza pedrobaeza deleted the 8.0-travis_lint branch October 19, 2015 00:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants