8000 Mention the handy `ignoreExtraKeys()` method by jordanlev · Pull Request #8675 · symfony/symfony-docs · GitHub
[go: up one dir, main page]
Skip to content

Mention the handy ignoreExtraKeys() method #8675

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
Merged
merged 1 commit into from
Nov 21, 2017
Merged

Mention the handy ignoreExtraKeys() method #8675

merged 1 commit into from
Nov 21, 2017

Conversation

jordanlev
Copy link
Contributor

No description provided.

@xabbuh
Copy link
Member
xabbuh commented Nov 16, 2017

We can apply this to the 2.7 branch too, right?

@jordanlev
Copy link
Contributor Author
jordanlev commented Nov 17, 2017 •
edited
Loading

A quick look through the history of the ArrayNodeDefinition.php file seems to show that this goes all the way back to 2.0 (at least):
https://github.com/symfony/symfony/blob/ec36ae7eda54d5407ecba77d57692df2a85b0fa6/src/Symfony/Component/Config/Definition/Builder/ArrayNodeDefinition.php#L234

@xabbuh xabbuh added this to the 2.7 milestone Nov 17, 2017
@weaverryan
Copy link
Member

Great job Jordan! Thank you!

@weaverryan weaverryan merged commit c175b0d into symfony:2.8 Nov 21, 2017
weaverryan added a commit that referenced this pull request Nov 21, 2017
@weaverryan
feature #8675 Mention the handy ignoreExtraKeys() method (jordanlev)
371738c 
This PR was merged into the 2.8 branch.

Discussion
----------

Mention the handy `ignoreExtraKeys()` method

Commits
-------

c175b0d Mention the handy `ignoreExtraKeys()` method
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xabbuh xabbuh xabbuh approved these changes

Assignees
No one assigned
Labels
Status: Reviewed
Projects
None yet
Milestone
2.7
Development

Successfully merging this pull request may close these issues.
4 participants
@jordanlev @xabbuh @weaverryan @carsonbot
0