Skip to content

[FrameworkBundle] Add info & example to auto_mapping config #34887

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
Dec 15, 2019
Merged

[FrameworkBundle] Add info & example to auto_mapping config #34887

merged 1 commit into from
Dec 15, 2019

Conversation

ogizanagi
Copy link
Contributor

Q A
Branch? 4.3
Bug fix? no
New feature? no
Deprecations? no
Tickets N/A
License MIT
Doc PR N/A

Backport part of #34707 and add example.

@@ -855,6 +855,11 @@ private function addValidationSection(ArrayNodeDefinition $rootNode)
->end()
->end()
->arrayNode('auto_mapping')
->info('A collection of namespaces for which auto-mapping will be enabled.')
Copy link
Member

Choose a reason for hiding this comment

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

That's describing the keys - should we describe the values also?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I changed this after @weaverryan 's suggestion in #34707 (comment) actually. Perhaps we can find something between, or just add info on the services node below.

@fabpot
Copy link
Member

fabpot commented Dec 15, 2019

Thank you @ogizanagi.

fabpot added a commit that referenced this pull request Dec 15, 2019
…fig (ogizanagi)

This PR was merged into the 4.3 branch.

Discussion
----------

[FrameworkBundle] Add info & example to auto_mapping config

| Q             | A
| ------------- | ---
| Branch?       | 4.3 <!-- see below -->
| Bug fix?      | no
| New feature?  | no <!-- please update src/**/CHANGELOG.md files -->
| Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files -->
| Tickets       | N/A <!-- prefix each issue number with "Fix #", if any -->
| License       | MIT
| Doc PR        | N/A

Backport part of #34707 and add example.

Commits
-------

8eb29a7 [FrameworkBundle] Add info & example to auto_mapping config
@fabpot fabpot merged commit 8eb29a7 into symfony:4.3 Dec 15, 2019
@ogizanagi ogizanagi deleted the fwb_val_auto_map_info branch December 15, 2019 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants