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.

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

Removed post-install-cmd php artisan optimize. #96

Merged
merged 1 commit into from
Nov 29, 2013

Conversation

flashingcursor
Copy link

Causes initial install to fail due to lack of required config (mail.php).

… to fail due to lack of required config (mail.php)
@snipe
Copy link
Owner

snipe commented Nov 29, 2013

I don't actually understand why this would happen though - the install command is run after you should have set up your mail config.

snipe added a commit that referenced this pull request Nov 29, 2013
Removed post-install-cmd php artisan optimize.
@snipe snipe merged commit bc16276 into snipe:develop Nov 29, 2013
@flashingcursor
Copy link
Author

Per the readme, you're doing a composer install before you config your mail and database to pull down dependancies.

@snipe
Copy link
Owner

snipe commented Nov 29, 2013

Yeah, I realized that after I asked the question :P It's merged.

@flashingcursor flashingcursor deleted the patch-1 branch November 29, 2013 21:16
@veen1981 veen1981 mentioned this pull request Jul 7, 2016
2 tasks
This was referenced Jul 19, 2016
@dooley1023 dooley1023 mentioned this pull request Feb 2, 2017
2 tasks
@katecruser katecruser mentioned this pull request Feb 8, 2017
2 tasks
@Mike1704 Mike1704 mentioned this pull request Mar 3, 2017
@ESWBitto ESWBitto mentioned this pull request Mar 16, 2017
2 tasks
@aaronlott aaronlott mentioned this pull request Jun 13, 2017
@hairysamurai hairysamurai mentioned this pull request Sep 12, 2017
@dotnvo dotnvo mentioned this pull request Aug 1, 2018
2 tasks
@rkayutkin rkayutkin mentioned this pull request Jun 14, 2019
2 tasks
@ITEnlighted ITEnlighted mentioned this pull request Jun 19, 2019
2 tasks
@anees78692 anees78692 mentioned this pull request Nov 29, 2019
@chandtech89 chandtech89 mentioned this pull request Sep 9, 2020
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