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

Debug container environment variables #11637

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Debug container environment variables #11637

wants to merge 1 commit into from

Conversation

dFayet
Copy link

@dFayet dFayet commented May 31, 2019

Hi,

This adds a little documentation for the env vars debug introduced by
symfony/symfony#27684

Fixes #11278

noniagriconomie
noniagriconomie reviewed May 31, 2019
View reviewed changes
Copy link
Contributor

@noniagriconomie noniagriconomie left a comment

Choose a reason for hiding this comment

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

nice addition

configuration/environment_variables.rst Outdated Show resolved Hide resolved
configuration/environment_variables.rst Show resolved Hide resolved
configuration/environment_variables.rst Show resolved Hide resolved
@javiereguiluz javiereguiluz added this to the 4.3 milestone Jun 3, 2019
@javiereguiluz javiereguiluz mentioned this pull request Jul 2, 2019
Merged
@javiereguiluz
Copy link
Member

@dFayet thanks a lot for this contribution! Sadly, since you contributed this we've changed the docs about configuration a lot, so this is "unmergeable". I tried to get your commit, rebase with the latest changes and make the needed changes ... but my Git skills are not as good, so I failed at doing that. Instead, I created #11865 based on your work.

I'm really sorry and I hope you can make more contributions in the future so we can merge them directly as we do with most of the pull requests. Thanks!

javiereguiluz added a commit that referenced this pull request Jul 3, 2019
@javiereguiluz
minor #11865 Debug container environment variables (javiereguiluz)
9a873fb 
This PR was squashed before being merged into the 4.3 branch (closes #11865).

Discussion
----------

Debug container environment variables

This replaces #11637 and fixes #11278.

Commits
-------

f5c0554 Debug container environment variables
@dFayet dFayet deleted the issue-11278 branch November 8, 2019 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@noniagriconomie noniagriconomie noniagriconomie left review comments

Assignees
No one assigned
Labels
DependencyInjection Status: Needs Review
Projects
None yet
Milestone
4.3
Development

Successfully merging this pull request may close these issues.
4 participants
@dFayet @javiereguiluz @noniagriconomie @carsonbot
Morty Proxy This is a proxified and sanitized view of the page, visit original site.