Skip to content

Move src/.bootstrap.php to config/bootstrap.php #499

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

Merged
1 commit merged into from
Nov 22, 2018
Merged

Move src/.bootstrap.php to config/bootstrap.php #499

1 commit merged into from
Nov 22, 2018

Conversation

nicolas-grekas
Copy link
Member

Q A
License MIT

I feel like this might be more natural. And we already have bundles.php there :)

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request passes validation.

@weaverryan
Copy link
Member

I like this, but I dislike changes :). Let’s do this, but quickly

@fabpot
Copy link
Member

fabpot commented Nov 22, 2018

Actually, is it really worth it?

@nicolas-grekas
Copy link
Member Author

The change is still cheap now, but won't be soon. Better do it and free us from explaining why this file (that's how this PR started :) )

@ghost ghost merged commit f54d6a4 into symfony:master Nov 22, 2018
ghost pushed a commit that referenced this pull request Nov 22, 2018
weaverryan added a commit to symfony/symfony-docs that referenced this pull request Nov 22, 2018
This PR was merged into the 4.1 branch.

Discussion
----------

Moving bootstrap.php to config

See symfony/recipes#499

Hopefully the last of such changes :)

Commits
-------

6b30ba1 moving bootstrap.php to config
@nicolas-grekas nicolas-grekas deleted the config.boot branch September 27, 2019 10:36
This pull request was closed.
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.

3 participants