Skip to content

[TwigBridge] deprecate the boolean object support trigger #17705

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
Feb 6, 2016

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Feb 5, 2016

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? yes
Tests pass? yes
Fixed tickets
License MIT
Doc PR

This reflects the changes done in the Yaml component in #17578.

@xabbuh xabbuh force-pushed the yaml-deprecation-twig-bridge branch from 531ee2f to 27243c6 Compare February 5, 2016 17:02
@xabbuh xabbuh changed the title [TwigBrdige] deprecate the boolean object support trigger [TwigBridge] deprecate the boolean object support trigger Feb 5, 2016
@fabpot
Copy link
Member

fabpot commented Feb 6, 2016

Thank you @xabbuh.

@fabpot fabpot merged commit 27243c6 into symfony:master Feb 6, 2016
fabpot added a commit that referenced this pull request Feb 6, 2016
…ger (xabbuh)

This PR was merged into the 3.1-dev branch.

Discussion
----------

[TwigBridge] deprecate the boolean object support trigger

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | yes
| Tests pass?   | yes
| Fixed tickets |
| License       | MIT
| Doc PR        |

This reflects the changes done in the Yaml component in #17578.

Commits
-------

27243c6 deprecate the boolean object support trigger
@xabbuh xabbuh deleted the yaml-deprecation-twig-bridge branch February 6, 2016 08:15
@nicolas-grekas
Copy link
Member

@xabbuh tests are failing with this :)

@xabbuh
Copy link
Member Author

xabbuh commented Feb 6, 2016

too bad :( see #17712 for the fix

xabbuh added a commit that referenced this pull request Feb 7, 2016
This PR was merged into the 3.1-dev branch.

Discussion
----------

[TwigBridge] fix default argument value

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #17705
| License       | MIT
| Doc PR        |

When deprecating boolean argument values, the default value must not be
`false`.

Commits
-------

beea61e [TwigBridge] fix default argument value
@fabpot fabpot mentioned this pull request May 13, 2016
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