-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Add parameters details for Heroku #4217
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
Comments
Indeed, environment variables are the best way to handle it (which is why the ParameterHandler supports mapping environment variables to parameters btw) |
Hi, There may be two possible sections for these things:
The Some more issues/tipps with Heroku:
Last point allows answering questions like "What should be the best practice to properly execute things like installing Assetic assets during the build?" (environment comes from
Please let me know. |
Well, it is explained in the documentation of the ParameterHandler for people susing it in other contexts. The symfony documentation is not the main documentation of the library.
We should at least mention it, as some things are indeed better using it currently (even though the goal would be that the official buildpack reaches parity on such support so that we can deprecate the CHH buildpack) |
Let's recap the things to do in this PR:
In my opinion, we just have to tackle 3 and 4 in |
That's specific to @CHH's now unmaintained buildpack. For the official Heroku buildpack, just use a |
That should never be a problem because your code shouldn't use it with the right environment set. You just need to make sure you |
@javiereguiluz Yes, still interested and motivated. Heroku's PHP support has improved a lot in the last few months so I'm going to check the open items. |
@javiereguiluz Mh, this is not an issue any more if you follow the best practice to use Heroku with a non-dev environment, e.g. if you set the prod environment as default. As Heroku uses |
The only open item for asset compiles is described here: https://devcenter.heroku.com/articles/php-support#build-behavior |
@dzuelke Thanks, already discovered it. I'll add a note about this and a short one about |
…u (bicpi) This PR was merged into the 2.3 branch. Discussion ---------- [Cookbook] Add warning about Composer dev deps on Heroku Q | A | ------------- | --- | Doc fix? | no | New docs? | no | Applies to | all | Fixed tickets | #4217 (partly) Commits ------- 3320b46 [Cookbook] Add warning about Composer dev deps on Heroku
This PR was merged into the 2.3 branch. Discussion ---------- [Cookbook] Custom compile steps on Heroku | Q | A | ------------- | --- | Doc fix? | no | New docs? | no | Applies to | all | Fixed tickets | #4217 (partially, list item 3) Commits ------- cb71ad0 [Cookbook] Custom compile steps on Heroku
Closing this - all the todos are "completed" - either have PR's or have open issues. Thanks! |
Hey guys!
A friend of mine was using Heroku with Symfony and got it all working nicely, except for handling parameters.yml. I realized that how parameters should be handled is not covered in our document. I believe that environment variables should be used.
The text was updated successfully, but these errors were encountered: