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

[9.0][PORT] port base location #187

Merged
merged 5 commits into from
Oct 21, 2015
Merged

Conversation

yvaucher
Copy link
Member

No description provided.

@yvaucher yvaucher changed the title 9.0 port base location [PORT][9.0] port base location Oct 14, 2015
@yvaucher yvaucher changed the title [PORT][9.0] port base location [9.0][PORT] port base location Oct 14, 2015
@pedrobaeza
Copy link
Member

Maybe you can take the occasion to make some tests for this module.

@charbeljc
Copy link

@pedrobaeza I would tend to merge as is, and focus on adding coverage reports first, so 👍

@pedrobaeza
Copy link
Member

The coverage reports are already there... via coveralls.io

@charbeljc
Copy link

hum, can't see it on this page ?

@pedrobaeza
Copy link
Member

It can be due to the high-traffic that is happening because the 9.0 branch migration, but it was working fine.

@yvaucher
Copy link
Member Author

@pedrobaeza Here we go all onchanges tested

@pedrobaeza
Copy link
Member

Thank you very much! Waiting for Travis result, 👍

@yvaucher
Copy link
Member Author

Travis is 💚

@pedrobaeza
Copy link
Member

Perfect, I merge then

pedrobaeza added a commit that referenced this pull request Oct 21, 2015
@pedrobaeza pedrobaeza merged commit 6703d33 into OCA:9.0 Oct 21, 2015
@yvaucher yvaucher deleted the 9.0-port-base_location branch October 21, 2015 09:33
@pedrobaeza pedrobaeza mentioned this pull request Oct 21, 2015
31 tasks
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

3 participants