Skip to content

Conversation

@frlan
Copy link
Contributor

@frlan frlan commented May 23, 2013

Without, postgreSQL based installations will not be able to migrated which prevents from being updated.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0%) when pulling 83f3cc433c7bf7b1d06af00ef594f9cd42e34db7 on frlan:doc/update/5.1-5.2/removing_explicit_deactivation_of_postgres into 76ffd11 on gitlabhq:master.

@Razer6
Copy link
Member

Razer6 commented May 24, 2013

+1 @bbodenmiller

…res based installations will not be able to migrated which prevents from being updated. Instead adding a switch for postgres and mysql.
@frlan
Copy link
Contributor Author

frlan commented May 24, 2013

I've just added a switch for mysql/postgres. I hope this make sense.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0%) when pulling 99f3bc3 on frlan:doc/update/5.1-5.2/removing_explicit_deactivation_of_postgres into bd6dfe7 on gitlabhq:master.

@bbodenmiller
Copy link
Contributor

I'm more a fan of #4053. Short simple to the point without duplicate code

@bbodenmiller
Copy link
Contributor

@senny #4068 addresses this PR.

@dzaporozhets
Copy link
Contributor

thank you. closed in favor of #4068

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants