Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.3k
Add documentation for therequire env processor#11313
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
wouterj left a comment
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.
Thanks for creating the documentation directly after submitting the PHP code!!
Once it's merged in the code, we should not forget to add a versionadded directive indicating when this feature was introduced.
Uh oh!
There was an error while loading.Please reload this page.
OskarStark left a comment
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.
Please add a versionadded directive like for the url env car processor
mpdude commentedApr 6, 2019
@OskarStark latest version added a versionadded notice |
This PR was merged into the 4.3-dev branch.Discussion----------[DIC] Add a `require` env var processor| Q | A| ------------- | ---| Branch? | master| Bug fix? | no| New feature? | yes| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets || License | MIT| Doc PR |symfony/symfony-docs#11313This adds a new `require` processor that will `require()` the PHP file given as input value and return the value `return`ed from that file. Leverages opcaching (yay!).#EUFOSSACommits-------03da3a22b1 Add a `require` env var processor
This PR was merged into the 4.3-dev branch.Discussion----------[DIC] Add a `require` env var processor| Q | A| ------------- | ---| Branch? | master| Bug fix? | no| New feature? | yes| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets || License | MIT| Doc PR |symfony/symfony-docs#11313This adds a new `require` processor that will `require()` the PHP file given as input value and return the value `return`ed from that file. Leverages opcaching (yay!).#EUFOSSACommits-------03da3a2 Add a `require` env var processor
wouterj commentedApr 7, 2019
Thanks again Matthias! |
…ude)This PR was squashed before being merged into the master branch (closes#11313).Discussion----------Add documentation for the `require` env processorDocumentation forsymfony/symfony#30897, discussed on the FOSSHackathon.Commits-------1011b42 Add documentation for the `require` env processor
Documentation forsymfony/symfony#30897, discussed on the FOSSHackathon.