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

Test pass #38

Closed
wants to merge 17 commits into from
Closed

Test pass #38

wants to merge 17 commits into from

Conversation

mileo
Copy link
Member

@mileo mileo commented Nov 11, 2015

FIX #37 #35

Good night!

@mileo
Copy link
Member Author

mileo commented Nov 11, 2015

Any one knows why we have this message, the oca_dependencies.txt seems to be ok

File "/srv/openerp/instances/openerp-oca-runbot/parts/odoo-extra/runbot/static/build/3119407-38-da5905/openerp/addons/base/module/module.py", line 393, in state_update
raise orm.except_orm(_('Error'), _("You try to install module '%s' that depends on module '%s'.\nBut the latter module is not available in your system.") % (module.name, dep.name,))
except_orm: ('Error', u"You try to install module 'account_fiscal_position_rule_purchase' that depends on module 'web_context_tunnel'.\nBut the latter module is not available in your system.")

@pedrobaeza
Copy link
Member

Can you squash the commit history a bit?

I have added the dependency to runbot and restart the build.

@mileo
Copy link
Member Author

mileo commented Nov 11, 2015

@pedrobaeza Done

@pedrobaeza
Copy link
Member

I still see 17 commits with very weak modifications like pep8 and so on. Please join also that.

@mileo
Copy link
Member Author

mileo commented Nov 11, 2015

@pedrobaeza You are talking about the Akretion's commits? cc @renatonlima @rvalyi

@rvalyi
Copy link
Member

rvalyi commented Nov 11, 2015

@pedrobaeza @mileo I'll squash the commits.
One reason why we broke down the commits so much is because many people are working on this in parallel or depending on this for further developments so we shared anything we could as quickly as possible. But no problem we can now squash some commits.

@rvalyi
Copy link
Member

rvalyi commented Nov 11, 2015

superseeded by #39 where I squashed 5 commits in @renatonlima work.

@rvalyi rvalyi closed this Nov 11, 2015
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.

4 participants