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.

Sign up for GitHub

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

Jump to bottom

remove git_data_dir #209

Merged
merged 5 commits into from
Mar 31, 2018
Merged

remove git_data_dir #209

merged 5 commits into from
Mar 31, 2018

Conversation

LongLiveCHIEF
Copy link
Contributor

  • removed and documented git_data_dir
  • removed the automatic execution of gitlab-rake gitlab:setup if
    service_manage = true and postgresql hash has configs but enable is undef || present.

Fixes #159
Fixes #35

@LongLiveCHIEF LongLiveCHIEF changed the title removed git_data_dir and db:setup remove git_data_dir and auto-execution of gitlab:setup if psql configured but not enabled Mar 31, 2018
@LongLiveCHIEF LongLiveCHIEF changed the title remove git_data_dir and auto-execution of gitlab:setup if psql configured but not enabled remove git_data_dir Mar 31, 2018
@LongLiveCHIEF
removed git_data_dir and db:setup
4e2f022 
- removed  and documented . Closes #159
- removed the automatic execution of  if
service_manage =true and postgresql hash has configs but enable is undef || present.
closes #35
@LongLiveCHIEF
Copy link
Contributor Author

I decided to split this into 2 PR's for the obvious reasons. I created #210 , and rebased this PR against that one. This PR requires #210 in order to be merged.

@bastelfreak
Copy link
Member

please rebase again against master

@LongLiveCHIEF
Copy link
Contributor Author

please rebase again against master

@bastelfreak now that you merged #210 to master, it has been rebased against master. 😃

@bastelfreak bastelfreak merged commit f8c27ff into master Mar 31, 2018
@bastelfreak bastelfreak deleted the 159-update-git_data_dirs branch March 31, 2018 21:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
backwards-incompatible
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@LongLiveCHIEF @bastelfreak