Skip to content

[Hackday] Update Heroku doc with SYMFONY_ENV param #4545

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

Closed
wants to merge 3 commits into from

Conversation

mxnr
Copy link

@mxnr mxnr commented Nov 29, 2014

Q A
Doc fix? [yes]
New docs? [no]
Applies to [2.5, 2.6, 2.7]
Fixed tickets #4217

@xabbuh
Copy link
Member

xabbuh commented Nov 29, 2014

This resolves #4217, right?

@mxnr
Copy link
Author

mxnr commented Nov 29, 2014

It looks like that, thanks @xabbuh! I update description.

.. code-block:: yaml

"post-install-cmd": [
"Incenteev\\ParameterHandler\\ScriptHandler::buildParameters",
Copy link
Contributor

Choose a reason for hiding this comment

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

is it really a good idea to show the complete content of the composer.json here?
this will become bad if we have to maintain this for different versions. i would prefer something like:

make sure you removed: ....
make sure you added: ....

Copy link
Member

Choose a reason for hiding this comment

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

I would indeed prefer to use ... to fold some irrelevant things.

Copy link
Contributor

Choose a reason for hiding this comment

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

This is no longer necessary, the underlying issue is now fixed in Symfony.

Copy link
Member

Choose a reason for hiding this comment

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

this section should then be removed from this PR to avoid confusion

@dzuelke
Copy link
Contributor

dzuelke commented Dec 4, 2014

Just realized my #4591 is a duplicate of this.

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.

6 participants