8000 Document `container.dumper.inline_class_loader` parameter · Issue #26269 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content

Document container.dumper.inline_class_loader parameter #26269

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
phansys opened this issue Feb 22, 2018 · 2 comments
Closed

Document container.dumper.inline_class_loader parameter #26269

phansys opened this issue Feb 22, 2018 · 2 comments

Comments

@phansys
Copy link
Contributor
phansys commented Feb 22, 2018
Q A
Bug report? no
Feature request? no
BC Break report? no
RFC? no
Symfony version 3.4/4.0

Parameter container.dumper.inline_class_loader should be documented. See https://twitter.com/nicolasgrekas/status/929032213815005184.

@nicolas-grekas
Copy link
Member
nicolas-grekas commented Feb 22, 2018

Note that the param is enabled by default for new projects that use either Flex or the Std edition.
I think the UPGRADE*.md files are a good fit.
Would you mind sending a PR against branch 3.4?

@phansys
Copy link
Contributor Author
phansys commented Feb 22, 2018

Sure, I'll try to create a PR today.

nicolas-grekas added a commit that referenced this issue Mar 2, 2018
…r` param at `UPGRADE-3.4.md` and `UPGRADE-4.0.md` (phansys)

This PR was merged into the 3.4 branch.

Discussion
----------

[Doc] Add entry for `container.dumper.inline_class_loader` param at `UPGRADE-3.4.md` and `UPGRADE-4.0.md`

|Q            |A     |
|---          |---   |
|Branch       |3.4   |
|Bug fix?     |no    |
|New feature? |no    |
|BC breaks?   |no    |
|Deprecations?|no    |
|Tests pass?  |yes   |
|Fixed tickets|#26269|
|License      |MIT   |
|Doc PR       |n/a   |

Commits
-------

d6e2b81 Add entry for `container.dumper.inline_class_loader` param at `UPGRADE-3.4.md` and `UPGRADE-4.0.md`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants
0