Skip to content

[ADD] base_setup: Analysis file + migration script - #796

Merged
hbrunn merged 2 commits into
OCA:10.0from
Tecnativa:10.0-base_setup
Apr 15, 2017
Merged

[ADD] base_setup: Analysis file + migration script#796
hbrunn merged 2 commits into
OCA:10.0from
Tecnativa:10.0-base_setup

Conversation

@pedrobaeza

Copy link
Copy Markdown
Member

No description provided.

@pedrobaeza

Copy link
Copy Markdown
Member Author

Failing while #795 is not merged.

@pedrobaeza

Copy link
Copy Markdown
Member Author

I include 6c4dd0a5807fe1e340f14231ecfd906da0faba19 meanwhile it's merged.

@pedrobaeza

Copy link
Copy Markdown
Member Author

@hbrunn, shouldn't coverage only be for migration scripts?

@hbrunn

hbrunn commented Apr 15, 2017

Copy link
Copy Markdown
Member

@pedrobaeza we miss https://github.com/OCA/OpenUpgrade/blob/9.0/.coveragerc - shouldn't this have been cherry picked from 9?

@pedrobaeza

Copy link
Copy Markdown
Member Author

Well, I don't know how that was treated for the initial 10.0 branch, but we can cherry-pick it in any moment. Preparing the PR...

@hbrunn

hbrunn commented Apr 15, 2017

Copy link
Copy Markdown
Member

an extra commit here doesn't hurt imho

@pedrobaeza

Copy link
Copy Markdown
Member Author

OK, you're right using the optimistic merging...

@hbrunn
hbrunn merged commit a71a42c into OCA:10.0 Apr 15, 2017
@pedrobaeza
pedrobaeza deleted the 10.0-base_setup branch April 15, 2017 18:11
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.

2 participants