YannickB / odoo-hosting

Other
64 stars 50 forks source link

[IMP][9.0] mqt: Add support for MQT builds #128

Closed lasley closed 7 years ago

lasley commented 7 years ago

This PR adds support for MQT builds. They are currently failing right now, but first step to 🍏 is 🔴 .

I'll be submitting subsequent PRs to fix the builds on a per-module basis, unless you would instead prefer one large one.

Changelog for this commit:

YannickB commented 7 years ago

Hello,

Well it was my plan to make a big PEP8 cleaning asap after Odoo Experience, I think it's time to do it.

It's best I do it myself, since I wrote most of the code I have better chance to not break anything. I'll do it starting from tomorrow, when my pycharm will stop yelling at me I'll accept your PR and we'll fix what's still missing. It should not take more than this week I think, hopefully only tomorrow.

lasley commented 7 years ago

Sounds like a plan, thanks @YannickB.

I'll tackle some of these errors on the tests side then. Looks like most are misses during v9 migrations - namely the sequences.

YannickB commented 7 years ago

Ok let's do this

YannickB commented 7 years ago

Ok the PEP8 is done, I merged the PR and make sure the repo is currently configured on travis. Let's continue the discussion here until travis is green.

lasley commented 7 years ago

Thanks @YannickB - I'm in the process of resolving the build errors now, you should have PR from me sometime today or tomorrow