-
-
Notifications
You must be signed in to change notification settings - Fork 811
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
Update bootstrap to check non-hidden config files #5748
Update bootstrap to check non-hidden config files #5748
Conversation
Huzzah! 🎊🎉 |
|
||
// An extra handler if a PHP bootstrap is provided, allow the bootstrap file to return | ||
// a pre-initialized Bolt Application rather than the config array. | ||
if (is_array($php)) { |
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.
PHP isn't defined for all paths … Travis go bye bye
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.
Travis the fascist.
} | ||
|
||
// An extra handler if a PHP bootstrap is provided, allow the bootstrap file to return | ||
// a pre-initialized Bolt Application rather than the config array. |
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.
This is already allowed below though?
<?php
return [
'application' => $myApp
];
Should bolt/composer-install be updated as well? |
As discussed in the dev-meeting: Merging in for 3.2, so we can test and bugfix before release! See https://github.com/bolt/bolt/wiki/Dev-meeting-2016-09-20 |
As discussed in #5729 Check for alternative non-dot versions of
.bolt.yml
and.bolt.php
.There's an additional minor improvement here and that allows a
bolt.php
file to just return a pre-constructed application rather than array. This is optional but simplifies the flow of php bootstrap files that will normally take care of creating an instance ofBolt\Application
.