Skip to content

[RFR] try to find out why database_cleaup messes up on travis#509

Closed
hbrunn wants to merge 2 commits into
OCA:9.0from
hbrunn:9.0-database-cleanup_tests-essay
Closed

[RFR] try to find out why database_cleaup messes up on travis#509
hbrunn wants to merge 2 commits into
OCA:9.0from
hbrunn:9.0-database-cleanup_tests-essay

Conversation

@hbrunn

@hbrunn hbrunn commented Aug 10, 2016

Copy link
Copy Markdown
Member

this is an exploratory PR, don't merge!

@hbrunn hbrunn added the invalid label Aug 10, 2016
which has weird side effects during testing. Take care
database_cleanup's tests don't mess up the following tests
@hbrunn

hbrunn commented Aug 10, 2016

Copy link
Copy Markdown
Member Author

@gurneyalex @StefanRijnhart as authors of #507 #508 would you agree if I PR reverting your change and replace it by this?

@gurneyalex

Copy link
Copy Markdown
Member

@hbrunn please do

Thanks for taking care of this 😸

👍 🍰

@hbrunn

hbrunn commented Aug 10, 2016

Copy link
Copy Markdown
Member Author

thanks, done

@hbrunn hbrunn closed this Aug 10, 2016
@StefanRijnhart

Copy link
Copy Markdown
Member

Thanks!

SiesslPhillip pushed a commit to grueneerde/OCA-server-tools that referenced this pull request Nov 20, 2024
Syncing from upstream OCA/server-tools (12.0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants