-
Notifications
You must be signed in to change notification settings - Fork 112
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
Add migration publishing, update config publishing and environmental variables #82
Add migration publishing, update config publishing and environmental variables #82
Conversation
Looks good, but is there any reason for removing the |
@codetheorist you'll break anything prior to v5.3 for not much gain. Just environment support is probably enough for this PR rather then removing code that supports Laravel 4 onwards. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add the code back for supporting Laravel 4 onwards and just give support for env config.
@@ -58,23 +58,20 @@ public function register() | |||
*/ | |||
public function boot() | |||
{ | |||
if (version_compare(Application::VERSION, '5.3', '>=')) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@codetheorist add this back.
@@ -9,15 +9,9 @@ class CreateSettingsTable extends Migration | |||
{ | |||
public function __construct() | |||
{ | |||
if (version_compare(Application::VERSION, '5.0', '>=')) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@codetheorist add this back.
@codetheorist any update? |
Issue #81: Add migration publishing to service provider
Fixes Issues #65 & #83: Have config file pull values from environmental variables
env()
helpers inconfig.php
to allow easier changing of important settings in a per-environment basis