Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Add documentation for the require 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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 7, 2019

Conversation

mpdude
Copy link
Contributor

@mpdude mpdude commented Apr 6, 2019

Documentation for symfony/symfony#30897, discussed on the FOSSHackathon.

@wouterj wouterj added the ⭐️ EU-FOSSA Hackathon https://symfony.com/blog/the-symfony-and-api-platform-hackathon-is-coming label Apr 6, 2019
@javiereguiluz javiereguiluz added the Waiting Code Merge Docs for features pending to be merged label Apr 6, 2019
@wouterj wouterj added this to the next milestone Apr 6, 2019
Copy link
Member

@wouterj wouterj left a 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.

configuration/environment_variables.rst Outdated Show resolved Hide resolved
Copy link
Contributor

@OskarStark OskarStark left a 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
Copy link
Contributor Author

mpdude commented Apr 6, 2019

@OskarStark latest version added a versionadded notice

symfony-splitter pushed a commit to symfony/dependency-injection that referenced this pull request Apr 7, 2019
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#11313

This 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!).

#EUFOSSA

Commits
-------

03da3a22b1 Add a `require` env var processor
nicolas-grekas added a commit to symfony/symfony that referenced this pull request Apr 7, 2019
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#11313

This 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!).

#EUFOSSA

Commits
-------

03da3a2 Add a `require` env var processor
@OskarStark OskarStark removed the Waiting Code Merge Docs for features pending to be merged label Apr 7, 2019
@wouterj
Copy link
Member

wouterj commented Apr 7, 2019

Thanks again Matthias!

@wouterj wouterj merged commit 1011b42 into symfony:master Apr 7, 2019
wouterj added a commit that referenced this pull request Apr 7, 2019
…ude)

This PR was squashed before being merged into the master branch (closes #11313).

Discussion
----------

Add documentation for the `require` env processor

Documentation for symfony/symfony#30897, discussed on the FOSSHackathon.

Commits
-------

1011b42 Add documentation for the `require` env processor
@mpdude mpdude deleted the docs-for-30897 branch April 8, 2019 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⭐️ EU-FOSSA Hackathon https://symfony.com/blog/the-symfony-and-api-platform-hackathon-is-coming Status: Reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.