Skip to content

Conversation

@ryasmi
Copy link
Member

@ryasmi ryasmi commented Aug 6, 2015

Ignoring the local config folder improves dev workflow by reducing the chance of merge conflicts and accidentally adding local configuration files to commits.

When this pull request is merged there is a one-time upgrade process (see below) that needs to be ran by anyone merging a branch or release with this fix.

  1. Run git fetch --tags.
  2. If this is your first time upgrading, run git checkout -b local.
  3. Run git add app/config/local; git commit -m 'Adds local config.'.
  4. If this is your first time upgrading, run git add -A; git commit -m 'Local changes.'.
  5. Run git merge TAG. Where TAG is the release (i.e. “v1.3.4”).
  6. Run git rm --cached app/config/local -r; git add app/config/local; git commit -m 'Fixes local config.'; php -r "readfile('https://getcomposer.org/installer');" | php; php composer.phar install.
  7. Run php artisan migrate --env=YOUR_ENV. Where YOUR_ENV should be replaced with your environment (likely to be local or production).

@daniel-abbey this is what you were testing on Wednesday.

@ryasmi ryasmi added this to the 1.8.0 milestone Aug 6, 2015
0xdeadbeefc0ffee pushed a commit that referenced this pull request Aug 7, 2015
Ignores local config folder to improve dev workflow
@0xdeadbeefc0ffee 0xdeadbeefc0ffee merged commit 69570d8 into develop Aug 7, 2015
@0xdeadbeefc0ffee 0xdeadbeefc0ffee mentioned this pull request Aug 7, 2015
@ryasmi ryasmi deleted the issue/config branch September 4, 2015 14:13
@0xdeadbeefc0ffee 0xdeadbeefc0ffee removed their assignment May 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants