[DI][Serializer] Fix missing de(normalizer|coder) autoconfig #22781

Merged
merged 1 commit into from May 21, 2017

Conversation

Projects
None yet
4 participants
@ogizanagi
Member

ogizanagi commented May 19, 2017

Q A
Branch? 3.3
Bug fix? yes(ish)
New feature? no (at least complete the 3.3 feature)
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets N/A
License MIT
Doc PR N/A

Just migrating an app to 3.3.0-RC1, I get surprised it wasn't already registered :)

@dunglas

This comment has been minimized.

Show comment
Hide comment
@dunglas

dunglas May 19, 2017

Member

Can you do the same for DecoderInterface?

Member

dunglas commented May 19, 2017

Can you do the same for DecoderInterface?

@ogizanagi

This comment has been minimized.

Show comment
Hide comment
@ogizanagi

ogizanagi May 19, 2017

Member

Oh, indeed, good catch! Done.

Member

ogizanagi commented May 19, 2017

Oh, indeed, good catch! Done.

@ogizanagi ogizanagi changed the title from [DI][Serializer] Fix missing denormalizer autoconfig to [DI][Serializer] Fix missing de(normalizer|coder) autoconfig May 19, 2017

@nicolas-grekas

👍 (failure unrelated)

@nicolas-grekas

This comment has been minimized.

Show comment
Hide comment
Member

nicolas-grekas commented May 21, 2017

Thank you @ogizanagi.

@nicolas-grekas nicolas-grekas merged commit 3c3a276 into symfony:3.3 May 21, 2017

1 of 3 checks passed

continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build failed
Details
fabbot.io Your code looks good.
Details

nicolas-grekas added a commit that referenced this pull request May 21, 2017

bug #22781 [DI][Serializer] Fix missing de(normalizer|coder) autoconf…
…ig (ogizanagi)

This PR was merged into the 3.3 branch.

Discussion
----------

[DI][Serializer] Fix missing de(normalizer|coder) autoconfig

| Q             | A
| ------------- | ---
| Branch?       | 3.3
| Bug fix?      | yes(ish)
| New feature?  | no (at least complete the 3.3 feature)
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | N/A
| License       | MIT
| Doc PR        | N/A

Just migrating an app to `3.3.0-RC1`, I get surprised it wasn't already registered :)

Commits
-------

3c3a276 [DI][Serializer] Fix missing de(normalizer|coder) autoconfig

@ogizanagi ogizanagi deleted the ogizanagi:fix/di/serializer_denormalizer_autoconfig branch May 21, 2017

@fabpot fabpot referenced this pull request May 29, 2017

Merged

Release v3.3.0 #22949

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment